[llvm-commits] arm mc: fix relocation of an alias to an alias

[Ivan Krasin]

Nick,

could you please take a quick look? I see your recent change to this file.

On Fri, Oct 7, 2011 at 9:49 AM, Jim Grosbach <grosbach at apple.com> wrote:
> This looks OK to me, but I'd really like if someone more familiar with the ELF bits would review the details.
>
> -Jim
>
> On Oct 5, 2011, at 5:33 PM, Ivan Krasin wrote:
>
>> Hi llvm team!
>>
>> The patch attached fixes the problem that
>> ARMELFObjectWriter::ExplicitRelSym didn't resolve aliases.
>> For some reason, the single alias works just fine, but the alias to
>> alias crashes llc with the following message:
>>
>> llc: /usr/local/google/home/krasin/llvm/include/llvm/MC/MCSymbol.h:111:
>> const llvm::MCSection &llvm::MCSymbol::getSection() const: Assertion
>> `isInSection() && "Invalid accessor!"' failed.
>>
>> The full background of this issue is available here:
>> http://code.google.com/p/nativeclient/issues/detail?id=2340
>>
>> OK to commit?
>>
>> krasin
>> <arm_reloc_alias_alias.patch>_______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>