[PATCH] D113125: [LoopVectorize] Propagate fast-math flags for VPInstruction

Rosie Sumpter via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 24 01:02:52 PST 2021


RosieSumpter marked an inline comment as done and an inline comment as not done.
RosieSumpter added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/VPlan.cpp:1214
+  O << " +";
+  if (isa<FPMathOperator>(getUnderlyingInstr()))
+    O << getUnderlyingInstr()->getFastMathFlags();
----------------
fhahn wrote:
> That change is not really related to the patch title/description. I think it would be good to either adjust the title or commit it separately.
Hi @fhahn, agreed - I split off this change into a separate commit before merging.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113125



More information about the llvm-commits mailing list