RFR: 8333393: PhaseCFG::insert_anti_dependences can fail to raise LCAs and to add necessary anti-dependence edges [v2]
Daniel Lundén
dlunden at openjdk.org
Tue Feb 25 14:54:36 UTC 2025
On Tue, 25 Feb 2025 11:23:28 GMT, Roberto Castañeda Lozano <rcastanedalo at openjdk.org> wrote:
>> Daniel Lundén has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Remove test that no longer reproduces the issue
>
> src/hotspot/share/opto/cfgnode.cpp line 2437:
>
>> 2435: worklist.push(this);
>> 2436: visited.set(this->_idx);
>> 2437: auto add_to_worklist = [&](Node* input) {
>
> Suggestion: rename to `maybe_add_to_worklist` to stress that it is a conditional addition.
Sure, now fixed.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/23691#discussion_r1969942203
More information about the hotspot-compiler-dev
mailing list