RFR: 8264954: unified handling for VectorMask object re-materialization during de-optimization [v2]
jbhateja at openjdk.java.net
Fri Apr 9 18:56:44 UTC 2021
On Fri, 9 Apr 2021 11:15:14 GMT, Vladimir Ivanov <vlivanov at openjdk.org> wrote:
>> Jatin Bhateja has updated the pull request incrementally with one additional commit since the last revision:
>> 8264954: Review comments resolution.
> Very nice! I like your idea to unify mask representation. It simplifies implementation a lot.
> But it also means there'll be 2 representations kept alive for masks with debug usages and burns 2 registers (speaking of x86, either 1 predicate + 1 vector registers on AVX512 or 2 vector regsiters on pre-AVX512). Do you see any problems with that? (It can be improved later if it turns out to be a problem in practice.)
Thanks @iwanowww , your comments are resolved.
More information about the hotspot-compiler-dev