[llvm] [X86] Don't rely on global -fp-contract=fast on X86 CodeGen tests (PR #158026)
Evgenii Kudriashov via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 15 17:49:44 PDT 2025
=?utf-8?q?Mikołaj_Piróg?= <mikolajpirog at gmail.com>,Pirog, Mikolaj
Maciej <mikolaj.maciej.pirog at intel.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/158026 at github.com>
================
@@ -10,10 +10,10 @@ define float @sqrt_ieee(float %f) #0 {
; CHECK-NEXT: {{ $}}
; CHECK-NEXT: [[COPY:%[0-9]+]]:fr32 = COPY $xmm0
; CHECK-NEXT: [[DEF:%[0-9]+]]:fr32 = IMPLICIT_DEF
- ; CHECK-NEXT: [[VSQRTSSr:%[0-9]+]]:fr32 = nofpexcept VSQRTSSr killed [[DEF]], [[COPY]], implicit $mxcsr
+ ; CHECK-NEXT: [[VSQRTSSr:%[0-9]+]]:fr32 = contract nofpexcept VSQRTSSr killed [[DEF]], [[COPY]], implicit $mxcsr
; CHECK-NEXT: $xmm0 = COPY [[VSQRTSSr]]
; CHECK-NEXT: RET 0, $xmm0
- %call = tail call float @llvm.sqrt.f32(float %f)
+ %call = tail call contract float @llvm.sqrt.f32(float %f)
----------------
e-kud wrote:
In this file we started adding `contract` where it wasn't previously regardless of `-fp-contract=fast` (see the absence of `contract` on the CHECK lines). Is it correct to put `contract` on them?
https://github.com/llvm/llvm-project/pull/158026
More information about the llvm-commits
mailing list