RFR: 8308339: AArch64: Remove extra `UseSVE` Predicate in ad file
Fei Gao
fgao at openjdk.org
Wed May 24 10:01:56 UTC 2023
On Wed, 24 May 2023 09:40:07 GMT, Andrew Haley <aph at openjdk.org> wrote:
> I appreciate the space saving, but this doesn't help maintainability. A predicate `UseSVE` is also a flag for the reader that this is SVE-only. It's also a simple rule to follow. Simple rules are good.
@theRealAph thanks for your review! How about moving these predicates as assertion lines to encode part? In this way, it could also work as indication.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/14112#issuecomment-1560817570
More information about the hotspot-compiler-dev
mailing list