[PATCH] D25017: [mips][msa] Range check MSA intrinsics with immediates

Vasileios Kalintiris via cfe-commits cfe-commits at lists.llvm.org
Tue Oct 18 04:35:36 PDT 2016


vkalintiris added inline comments.


================
Comment at: lib/Sema/SemaChecking.cpp:1633-1637
+  if (!m)
+    return SemaBuiltinConstantArgRange(TheCall, i, l, u);
+
+    return SemaBuiltinConstantArgRange(TheCall, i, l, u) ||
+           SemaBuiltinConstantArgMultiple(TheCall, i, m);
----------------
Also, can you fix the indentation here and check with clang-format for long lines in the code?


https://reviews.llvm.org/D25017





More information about the cfe-commits mailing list