[PATCH] D16207: [Reassociate] : Make sure when we are optimizing an instruction, it's operands have already been canonicalized

Aditya Nandakumar via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 14 17:01:34 PST 2016


aditya_nandakumar created this revision.
aditya_nandakumar added reviewers: majnemer, mcrosier, echristo, qcolombet.
aditya_nandakumar added a subscriber: llvm-commits.
aditya_nandakumar set the repository for this revision to rL LLVM.

This is a continuation (I couldn't update the phab review) from http://reviews.llvm.org/D6995 (Original Patch by Mehdi)

Before we Optimize instruction, make sure all instructions that are in the redo list and are before the current instruction have been optimized.

Repository:
  rL LLVM

http://reviews.llvm.org/D16207

Files:
  lib/Transforms/Scalar/Reassociate.cpp
  test/Transforms/Reassociate/prev_insts_canonicalized.ll
  test/Transforms/Reassociate/reassoc-intermediate-fnegs.ll
  test/Transforms/Reassociate/xor_reassoc.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16207.44950.patch
Type: text/x-patch
Size: 7257 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160115/53f03ef2/attachment.bin>


More information about the llvm-commits mailing list