[PATCH] D131064: [AArch64] Alter arm_sve.h to be target-based, not preprocessor based.

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 3 06:28:15 PDT 2022


simon_tatham accepted this revision.
simon_tatham added a comment.
This revision is now accepted and ready to land.

I love it! I'm generally frustrated by any software that says "what even is this, I've never heard of it" when it could so easily change it to "I have heard of this but it's not allowed in this particular context because reason". If nothing else, it means you get different error messages for "misspelled an intrinsic" and "used one from SVE version n+1".


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

https://reviews.llvm.org/D131064



More information about the llvm-commits mailing list