[PATCH] D12096: [InstCombineAddSub opportunities]: More opportunities to factorize FAdd/FSub when unsafeAlgebra is present for Inst

[Aditya Nandakumar via llvm-commits]

aditya_nandakumar updated this revision to Diff 32774.
aditya_nandakumar marked an inline comment as done.
aditya_nandakumar added a comment.

Updated comments.
Moved the checking of operands directly to if statement instead of assigning to bool and checking.


http://reviews.llvm.org/D12096

Files:
  lib/Transforms/InstCombine/InstCombineAddSub.cpp
  test/Transforms/InstCombine/FAddFSubAssociativeFactorize.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D12096.32774.patch
Type: text/x-patch
Size: 18012 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150821/4a62b25c/attachment.bin>