[PATCH] D127406: llvm-reduce: Support replacing FP values with 1.0

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks accepted this revision.
aeubanks added a comment.
This revision is now accepted and ready to land.

lg if test comments are addressed



================
Comment at: llvm/test/tools/llvm-reduce/remove-operands-fp.ll:42-44
+declare float @llvm.fma.f32(float, float, float)
+declare <2 x float> @llvm.fma.v2f32(<2 x float>, <2 x float>, <2 x float>)
+
----------------
these aren't used


================
Comment at: llvm/test/tools/llvm-reduce/remove-operands-fp.ll:48
+  %fadd0 = fadd float %arg0, %arg1
+  store float %fadd0, float* undef
+
----------------
these store aren't relevant to the test


================
Comment at: llvm/test/tools/llvm-reduce/remove-operands-fp.ll:56
+
+  %fadd3 = fadd <2 x float> %arg2, %arg3
+  store <2 x float> %fadd3, <2 x float>* undef
----------------
was this meant to be `zeroinitializer`?


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

https://reviews.llvm.org/D127406