[AArch64] provide strictfp attributes in test file
authorSimon Wallis <simon.wallis2@arm.com>
Thu, 26 Aug 2021 15:56:26 +0000 (16:56 +0100)
committerSimon Wallis <simon.wallis2@arm.com>
Thu, 26 Aug 2021 15:56:43 +0000 (16:56 +0100)
A post-commit review comment on  https://reviews.llvm.org/D107452 pointed out that
https://llvm.org/docs/LangRef.html
says:
"In a function that uses the constrained intrinsics the strictfp attribute is required on all function calls."

Although there are several files across several test directories which don't follow this guidance, it is straightforward to provide this attribute.

Reviewed By: kpn

Differential Revision: https://reviews.llvm.org/D107567

llvm/test/CodeGen/AArch64/fptosi-strictfp.ll

index 2407758..62c5d4f 100644 (file)
@@ -2,22 +2,24 @@
 target datalayout = "e-m:e-i8:8:32-i16:16:32-i64:64-i128:128-n32:64-S128"
 target triple = "aarch64-arm-none-eabi"
 
-define i128 @test_fixtfti(fp128 %ld) {
+define i128 @test_fixtfti(fp128 %ld) #0 {
 ; CHECK-LABEL: test_fixtfti:
 ; CHECK: bl    __fixtfti
 entry:
-  %conv = call i128 @llvm.experimental.constrained.fptosi.i128.f128(fp128 %ld, metadata !"fpexcept.strict")
+  %conv = call i128 @llvm.experimental.constrained.fptosi.i128.f128(fp128 %ld, metadata !"fpexcept.strict") #0
   ret i128 %conv
 }
 
 declare i128 @llvm.experimental.constrained.fptosi.i128.f128(fp128, metadata)
 
-define i128 @test_fixtftu(fp128 %ld) {
+define i128 @test_fixtftu(fp128 %ld) #0 {
 ; CHECK-LABEL: test_fixtftu:
 ; CHECK: bl    __fixunstfti
 entry:
-  %conv = call i128 @llvm.experimental.constrained.fptoui.i128.f128(fp128 %ld, metadata !"fpexcept.strict")
+  %conv = call i128 @llvm.experimental.constrained.fptoui.i128.f128(fp128 %ld, metadata !"fpexcept.strict") #0
   ret i128 %conv
 }
 
 declare i128 @llvm.experimental.constrained.fptoui.i128.f128(fp128, metadata)
+
+attributes #0 = { strictfp }