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

[Daniel Berlin via llvm-commits]

I'm confused ;-)


The last status i saw was: "I'll try seeing what little needs to be done to
converge in the couple cases that I have (one above) and a superficial look
at why the second reassociate improves codegen."

I saw no update on that ;-)

and
"Is there already an implementation for global reassociate?"

Which i missed.

The answer to this is yes". N-Ary reassociate is already in tree, and
making it "better" shouldn't be that hard (if it turns out to be hard,
great, let's hack up local reassociate if we have to)


On Thu, Nov 12, 2015 at 2:48 PM, Owen Anderson <resistor at mac.com> wrote:

> resistor added a subscriber: resistor.
> resistor added a comment.
>
> Ping?  Was there any further feedback on this?  We're seeing pretty
> horrible regressions caused by this.
>
>
> Repository:
>   rL LLVM
>
> http://reviews.llvm.org/D12345
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151112/50cfd849/attachment.html>