[PATCH] D61262: [AArch64] Implement MC support for Scalable Vector Extension 2 (SVE2)

[Javed Absar via Phabricator via llvm-commits]

javed.absar added inline comments.


================
Comment at: lib/Target/AArch64/AArch64SchedExynosM1.td:27
 
-  list<Predicate> UnsupportedFeatures = [HasSVE];
+  list<Predicate> UnsupportedFeatures = [HasSVE, HasSVE2, HasSVE2AES,
+                                         HasSVE2SM4, HasSVE2SHA3,
----------------
Instead of repeating this list everywhere you can at top level : 

class AArch64UnsupportedF { list<Predicate> F; }
def  SVEUnsupported F : AArch64UnsupportedF {
    let F = [HasSVE, HasSVE2, ... ];
}
  
and then in each AArch64SchedX.td ::
 list<Predicate> UnsupportedFeatures = SVEUnsupported.F;

TableGen curently does not have globals otherwise it would be simpler,  but this idiom works



Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61262/new/

https://reviews.llvm.org/D61262