[PATCH] D18499: [ELF] - Implemented basic location counter support.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 15 06:41:10 PDT 2016


grimar added a comment.

About http://reviews.llvm.org/D19107 and SectionOrder vector.
What about landing this and eliminating the SectionOrder at all ?

At fact when we have Locations structure, SectionOrder is just a subset. We can use
Locations instead to sort sections in compareSections.

The only problem then - that fixupLocations() should be called much earlier then. 
I think previous iteration was better for such purposes.


http://reviews.llvm.org/D18499





More information about the llvm-commits mailing list