https://llvm.org/bugs/show_bug.cgi?id=17305
Sanjay Patel <spatel+l...@rotateright.com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution|--- |FIXED
--- Comment #4 from Sanjay Patel <spatel+l...@rotateright.com> ---
Reassociation is now done in the machine combiner pass so it can be
register-pressure-aware.
The code produced after:
http://llvm.org/viewvc/llvm-project?view=revision&revision=241871
...is equivalent to that in comment 2.
It's still not the perfect binary tree of ops because that would require a
potentially big leap in compile time, but this is probably as good as it gets.
Other relevant commits:
http://llvm.org/viewvc/llvm-project?view=revision&revision=239486
http://llvm.org/viewvc/llvm-project?view=revision&revision=240361
http://llvm.org/viewvc/llvm-project?view=revision&revision=241515
--
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
LLVMbugs mailing list
LLVMbugs@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvmbugs
