[PATCH] D26543: Fix for lost FastMathFlags in SLPVectorizer

Michael Zolotukhin via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 11 10:25:01 PST 2016


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

LGTM! I wonder if there are other types of instructions that can have flags (except BinaryOperation and CmpInstruction), which we need to check as well.

Michael



================
Comment at: llvm/test/Transforms/SLPVectorizer/X86/propagate_ir_flags.ll:351
  
+attributes #1 = { "target-features"="+avx" }
+
----------------
Can we put that in the bottom of the file?


https://reviews.llvm.org/D26543





More information about the llvm-commits mailing list