[PATCH] [Layout] Assign ordinals in resolution order

Rui Ueyama ruiu at google.com
Tue Oct 8 22:47:45 PDT 2013


On Tue, Oct 8, 2013 at 10:37 PM, Shankar Easwaran
<shankare at codeaurora.org>wrote:

> On 10/9/2013 12:31 AM, Rui Ueyama wrote:
>
>>    Really? Then how would you handle --{start,end}-group?
>>
> Its already handled in resetNextIndex. --start-group/--end-group works
> with ELF already with my previous change!.
>

I added "assert(!nextFile->hasOrdinal());" at line Resolver.cpp:302, then
many tests failed with the assertion error. Why?
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20131008/10540755/attachment.html>


More information about the llvm-commits mailing list