[code-reflection] Integrated: Fix SSABraun bug and add SSA tests
Ruby Chen
duke at openjdk.org
Thu Sep 11 18:25:55 UTC 2025
On Tue, 2 Sep 2025 20:30:58 GMT, Ruby Chen <duke at openjdk.org> wrote:
> Fix a bug in SSABraun where, in `tryRemoveTrivialPhi()`, a phi stored in `same` that is later deleted in a recursive `tryRemoveTrivialPhi()` call is still returned despite being deleted.
>
> Add five tests to TestSSA: `deadCode(), ifelseLoopNested(), violaJones(), binarySearch(),` and `quicksort()`. `violaJones()` is inspired by the method `findFeaturesKernel` in HAT kernel ViolaJones, which is the bug first presented itself.
This pull request has now been integrated.
Changeset: ab94fe70
Author: Ruby Chen <ruby.r.chen at gmail.com>
Committer: Paul Sandoz <psandoz at openjdk.org>
URL: https://git.openjdk.org/babylon/commit/ab94fe70c5fd3522469ff12a74e27d1ad4157002
Stats: 176 lines in 2 files changed: 175 ins; 0 del; 1 mod
Fix SSABraun bug and add SSA tests
Reviewed-by: psandoz
-------------
PR: https://git.openjdk.org/babylon/pull/542
More information about the babylon-dev
mailing list