[PATCH] D12345: [Reassociate]: Add intermediate subtract instructions created while negating to be redone later for more reassociate opportunities

hfinkel@anl.gov via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 10 14:27:10 PST 2015


hfinkel added a subscriber: hfinkel.
hfinkel added a comment.

In http://reviews.llvm.org/D12345#301762, @aditya_nandakumar wrote:

> Thanks Owen. Sorry about not updating this. This patch caused some regressions on some tests and I hadn't fully figured out/isolated the regression.


What kinds of regressions? Can you replicate these by running Reassociate twice in an otherwise-standard pipeline? We really should get this figured out.

Danny, are you proposing that we extend N-ary reassociation to work on floating-point values?


Repository:
  rL LLVM

http://reviews.llvm.org/D12345





More information about the llvm-commits mailing list