On Tue, 28 Sep 2021 10:01:43 GMT, Claes Redestad <redes...@openjdk.org> wrote:
>> Not too much work. I recently introduced platform-specific `matcher_*.hpp` >> files, so since then adding a boolean constant is easy (no need to muck with >> the .ad files). > > Does the changes in 9800a99 resolve your concerns? In principle yes, but shouldn't the condition read: if (!Matcher::match_rule_supported(Op_EncodeISOArray) || !Matcher::supports_encode_ascii_array) return false; I.e. the intrinisc is supported if both conditions are true and not supported if either one of them is false? ------------- PR: https://git.openjdk.java.net/jdk/pull/5621