[llvm-commits] [llvm] r166051 - /llvm/trunk/lib/MC/ELFObjectWriter.cpp

[Rafael Espíndola]

ah, thanks!

On 17 October 2012 15:58, Michael Liao <michael.liao at intel.com> wrote:
> There's no functional change. Just clean up the code to remove
> target-specific code in target independent MC part.
>
> - Michael
>
> On Wed, 2012-10-17 at 15:55 -0400, Rafael Espíndola wrote:
>> is it possible to add a testcase?
>>
>> On 16 October 2012 15:49, Michael Liao <michael.liao at intel.com> wrote:
>> > Author: hliao
>> > Date: Tue Oct 16 14:49:51 2012
>> > New Revision: 166051
>> >
>> > URL: http://llvm.org/viewvc/llvm-project?rev=166051&view=rev
>> > Log:
>> > Check .rela instead of ELF64 for the compensation vaue resetting
>> >
>> >
>> > Modified:
>> >     llvm/trunk/lib/MC/ELFObjectWriter.cpp
>> >
>> > Modified: llvm/trunk/lib/MC/ELFObjectWriter.cpp
>> > URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/MC/ELFObjectWriter.cpp?rev=166051&r1=166050&r2=166051&view=diff
>> > ==============================================================================
>> > --- llvm/trunk/lib/MC/ELFObjectWriter.cpp (original)
>> > +++ llvm/trunk/lib/MC/ELFObjectWriter.cpp Tue Oct 16 14:49:51 2012
>> > @@ -733,8 +733,7 @@
>> >        Index = -1;
>> >      }
>> >      Addend = Value;
>> > -    // Compensate for the addend on i386.
>> > -    if (is64Bit())
>> > +    if (hasRelocationAddend())
>> >        Value = 0;
>> >    }
>> >
>> >
>> >
>> > _______________________________________________
>> > llvm-commits mailing list
>> > llvm-commits at cs.uiuc.edu
>> > http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>