[PATCH] D19924: Representing bundle locked groups as fragments in MCAssembler

Petr Hosek via llvm-commits llvm-commits at lists.llvm.org
Fri May 6 11:20:37 PDT 2016


phosek added a comment.

I'm perfectly happy with the removal of the relax-all related logic. This was a workaround to reduce the memory usage in constrained environments like PNaCl (where we only have 768MB of address space available), but at the cost of producing fairly suboptimal instruction sequences in some cases. We've recently switched from llc to Subzero for PNaCl and so this workaround is no longer needed.

The new implementation looks significantly cleaner, introducing the new fragment type for bundle handling seems like the right way of implementing bundling.


Repository:
  rL LLVM

http://reviews.llvm.org/D19924





More information about the llvm-commits mailing list