[PATCH] D73194: [FPEnv][ARM] Add lowering of STRICT_FSETCC and STRICT_FSETCCS

Thu Jan 30 04:41:34 PST 2020

dmgreen added a comment.

Is it possible for arm_cmpfpe have both a chain and a glue? I presume it's the cmp that would need the chain, not anything else that gets expanded?

================
Comment at: llvm/test/CodeGen/ARM/fp-intrinsics.ll:73
 ; CHECK-NOSP: bl __aeabi_f2uiz
-; CHECK-SP: vcvt.u32.f32
+; FIXME-CHECK-SP: vcvt.u32.f32
 define i32 @fptoui_f32(float %x) #0 {
----------------
This is the thing that only works when dp is enabled? What was the deal there?

Am I right in saying that they are not incorrect, just inefficient?

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D73194/new/

https://reviews.llvm.org/D73194