[PATCH] D26575: Fix for lost FastMathFlags in SLPVectorizer (intrinsic calls)

Vyacheslav Klochkov via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 15 17:13:04 PST 2016


v_klochkov added a comment.

Michael, thank you for the quick code-review!



================
Comment at: llvm/test/Transforms/SLPVectorizer/X86/propagate_ir_flags.ll:413
+  %call1 = tail call fast double @llvm.fabs.f64(double %load1) nounwind readnone
+  %call2 = tail call fast double @llvm.fabs.f64(double %load1) nounwind readnone
+
----------------
mzolotukhin wrote:
> Should it be `%load2`?
Yes, it was going to be %load2. 
Even though the test worked well as is, I fixed it.


Repository:
  rL LLVM

https://reviews.llvm.org/D26575





More information about the llvm-commits mailing list