[lld] r221427 - [ELF] Implement isLittleEndian for all architectures

Rafael Avila de Espindola rafael.espindola at gmail.com
Thu Nov 6 09:18:44 PST 2014


We have one in llvm.

Sent from my iPhone

> On Nov 6, 2014, at 11:49, Shankar Easwaran <shankare at codeaurora.org> wrote:
> 
>> On 11/6/2014 9:52 AM, Charlie Turner wrote:
>> Hi Shankar,
>> 
>>> No. There is no reason too -- the CPU does not allow endian switching
>> unlike ARM.
>> 
>> That's why this method wasn't pure virtual to start with. This distinction
>> isn't relevant for all architectures.
> I didnt want to make a blanket function, that returns true for all architectures. Its not good to have a default value for endianness.
> 
> Shankar Easwaran
> 
> -- 
> Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by the Linux Foundation
> 
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list