[PATCH] D83111: [X86-64] Support Intel AMX Intrinsic

Xiang Zhang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 3 22:52:17 PDT 2020


xiangzhangllvm marked an inline comment as done.
xiangzhangllvm added inline comments.


================
Comment at: clang/lib/Sema/SemaChecking.cpp:3613
+                                    ArrayRef<int> ArgNums,
+                                    int Low, int High) {
+  for (int ArgNum : ArgNums) {
----------------
craig.topper wrote:
> Why are Low/High arguments? Aren't they always 0 and 7.
Yes, they are 0 and 7, keep the Low and High will make it easy to read.


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

https://reviews.llvm.org/D83111





More information about the llvm-commits mailing list