RFR: 8288883: C2: assert(allow_address || t != T_ADDRESS) failed after JDK-8283091
Fei Gao
fgao at openjdk.org
Fri Jul 8 01:48:41 UTC 2022
On Thu, 7 Jul 2022 18:54:06 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:
> In which call to `adjust_alignment_for_type_conversion()` you got AddP node? Should we add checks there too?
Thanks for your review, @vnkozlov .
When we called `adjust_alignment_for_type_conversion()` in `SuperWord::follow_def_uses()`, https://github.com/openjdk/jdk/blob/3f1174aa4709aabcfde8b40deec88b8ed466cc06/src/hotspot/share/opto/superword.cpp#L1525, we got AddP node. In this function, we also call `stmts_can_pack()` on the next line, which has checks to prevent unwanted pairs, https://github.com/openjdk/jdk/blob/3f1174aa4709aabcfde8b40deec88b8ed466cc06/src/hotspot/share/opto/superword.cpp#L1202. Maybe we don't have to add one more. WDYT?
-------------
PR: https://git.openjdk.org/jdk/pull/9391
More information about the hotspot-compiler-dev
mailing list