[PATCH] D104255: [InterleaveAccess] Copy fast math flags when adjusting binary operators in interleave access pass

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 15 01:08:07 PDT 2021


dmgreen added inline comments.


================
Comment at: llvm/lib/CodeGen/InterleavedAccessPass.cpp:411
         SVI->getName(), SVI);
-    Value *NewBI = BinaryOperator::Create(BI->getOpcode(), NewSVI1, NewSVI2,
-                                          BI->getName(), SVI);
+    BinaryOperator *NewBI = BinaryOperator::Create(BI->getOpcode(), NewSVI1,
+                                                   NewSVI2, BI->getName(), SVI);
----------------
jsji wrote:
> No sure, but maybe we can use `CreateWithCopiedFlags` directly?
Oh yeah, that's what I was looking for! I had to add an version with an InsertBefore point, too.


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

https://reviews.llvm.org/D104255



More information about the llvm-commits mailing list