[llvm-commits] Fix for ArchiveReader

[Rafael Avila de Espindola]

On 11-03-04 10:26 AM, Joe Abbey wrote:
...


> The defect is the 'offset' value saved to the 'modules' map. The
> 'modules' map should store the physical file offset of the loaded
> module (which is "offset + firstFileOffset", instead of the "offset"),
> the distance of the loaded module member from the first member in the
> archive file.

A testcase would be nice. The archive files that llvm produces are a bit 
funny, if I remember correctly they have both a regular symbol table and 
a table of the IL. I assume the problem you found is while reading an 
archive produced by 'ar rc' (without the s). If this fixes it, it is 
fine. Can you use the system ar + llvm-nm to produce a test?

> Cheers,
>
> Joe
>

Thanks,
Rafael