[PATCH] D63445: [Reassociate] Handle unary FNeg in the Reassociate pass

Cameron McInally via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 18 07:19:39 PDT 2019


cameron.mcinally marked 3 inline comments as done.
cameron.mcinally added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/Reassociate.cpp:1974
 
   // User must be a binary operator with one or more uses.
   Instruction *User = I->user_back();
----------------
craig.topper wrote:
> Update comment
Bah, these two changes weren't supposed to be a part of this patch. Just an oversight and will update...


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

https://reviews.llvm.org/D63445





More information about the llvm-commits mailing list