[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