[llvm] r366280 - ARM: Fix missing immarg for space intrinsic

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 16 15:41:38 PDT 2019


Author: arsenm
Date: Tue Jul 16 15:41:38 2019
New Revision: 366280

URL: http://llvm.org/viewvc/llvm-project?rev=366280&view=rev
Log:
ARM: Fix missing immarg for space intrinsic

Modified:
    llvm/trunk/include/llvm/IR/IntrinsicsARM.td
    llvm/trunk/test/Verifier/ARM/intrinsic-immarg.ll

Modified: llvm/trunk/include/llvm/IR/IntrinsicsARM.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/IR/IntrinsicsARM.td?rev=366280&r1=366279&r2=366280&view=diff
==============================================================================
--- llvm/trunk/include/llvm/IR/IntrinsicsARM.td (original)
+++ llvm/trunk/include/llvm/IR/IntrinsicsARM.td Tue Jul 16 15:41:38 2019
@@ -19,7 +19,7 @@ let TargetPrefix = "arm" in {  // All in
 // A space-consuming intrinsic primarily for testing ARMConstantIslands. The
 // first argument is the number of bytes this "instruction" takes up, the second
 // and return value are essentially chains, used to force ordering during ISel.
-def int_arm_space : Intrinsic<[llvm_i32_ty], [llvm_i32_ty, llvm_i32_ty], []>;
+def int_arm_space : Intrinsic<[llvm_i32_ty], [llvm_i32_ty, llvm_i32_ty], [ImmArg<0>]>;
 
 // 16-bit multiplications
 def int_arm_smulbb : GCCBuiltin<"__builtin_arm_smulbb">,

Modified: llvm/trunk/test/Verifier/ARM/intrinsic-immarg.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Verifier/ARM/intrinsic-immarg.ll?rev=366280&r1=366279&r2=366280&view=diff
==============================================================================
--- llvm/trunk/test/Verifier/ARM/intrinsic-immarg.ll (original)
+++ llvm/trunk/test/Verifier/ARM/intrinsic-immarg.ll Tue Jul 16 15:41:38 2019
@@ -100,3 +100,12 @@ define void @mcrr2(i32 %arg0, i32 %arg1,
   call void @llvm.arm.mcrr2(i32 0, i32 1, i32 2, i32 3, i32 %arg4)
   ret void
 }
+
+declare i32 @llvm.arm.space(i32, i32) nounwind
+define i32 @space(i32 %arg0, i32 %arg1) {
+  ; CHECK: immarg operand has non-immediate parameter
+  ; CHECK-NEXT: i32 %arg0
+  ; CHECK-NEXT: call i32 @llvm.arm.space(i32 %arg0, i32 %arg1)
+  %space = call i32 @llvm.arm.space(i32 %arg0, i32 %arg1)
+  ret i32 %space
+}




More information about the llvm-commits mailing list