arsenm added inline comments.
================
Comment at: clang/include/clang/Basic/BuiltinsLoongArch.def:20-23
TARGET_BUILTIN(__builtin_loongarch_dbar, "vIUi", "nc", "")
+TARGET_BUILTIN(__builtin_loongarch_ibar, "vIUi", "nc", "")
+TARGET_BUILTIN(__builtin_loongarch_break, "vIUi", "nc", "")
+TARGET_BUILTIN(__builtin_loongarch_syscall, "vIUi", "nc", "")
----------------
These all have I parameters but the underlying intrinsic wasn't marked with
immarg
================
Comment at: clang/test/CodeGen/LoongArch/intrinsic-error.c:21
+ __builtin_loongarch_ibar(-1); // expected-error {{argument value 4294967295
is outside the valid range [0, 32767]}}
+}
+
----------------
Should also test non-constant cases
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D138094/new/
https://reviews.llvm.org/D138094
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
