[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