[PATCH] D77540: [PATCH] [ARM]: Armv8.6-a Matrix Mul Asm and Intrinsics Support

Luke Geeson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Apr 8 07:02:31 PDT 2020


LukeGeeson added inline comments.


================
Comment at: clang/lib/Basic/Targets/AArch64.h:39
   bool HasTME;
+  unsigned HasMatMul;
 
----------------
DavidSpickett wrote:
> Why is this feature a number for AArch64, does >1 mean something?
It seems this was an artifact from how this is handled in ARM.h where they do have special meaning. I'll modify this to be boolean in line with the others in AArch64.h


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

https://reviews.llvm.org/D77540





More information about the cfe-commits mailing list