[llvm] r239071 - [bpf] add big- and host- endian support

Alexei Starovoitov alexei.starovoitov at gmail.com
Thu Jun 4 15:55:03 PDT 2015


On Thu, Jun 4, 2015 at 3:37 PM, Eric Christopher <echristo at gmail.com> wrote:
>
>> What the patch does:
>> -march=bpf -> host endian
>> -march=bpf_le -> little endian
>> -march=bpf_be -> big endian
>>
>
> So, part of why I was going to review the Triple part of it is that the
> naming convention for bpf big and little endian explicit triples doesn't
> match the same convention for every other target. So, what references do you
> have for these outside of llvm? I'd prefer to copy the existing nomenclature
> if there's no compelling reason otherwise.

I've copied aarch64, aarch64_be convention.
You mean you prefer without underscore?
I think bpfle, bpfbe are less readable.
Also there is no 'default' endianness.
So what do you have in mind specificialy?



More information about the llvm-commits mailing list