<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 4, 2015 at 3:55 PM Alexei Starovoitov <<a href="mailto:alexei.starovoitov@gmail.com">alexei.starovoitov@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Jun 4, 2015 at 3:37 PM, Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>> wrote:<br>
><br>
>> What the patch does:<br>
>> -march=bpf -> host endian<br>
>> -march=bpf_le -> little endian<br>
>> -march=bpf_be -> big endian<br>
>><br>
><br>
> So, part of why I was going to review the Triple part of it is that the<br>
> naming convention for bpf big and little endian explicit triples doesn't<br>
> match the same convention for every other target. So, what references do you<br>
> have for these outside of llvm? I'd prefer to copy the existing nomenclature<br>
> if there's no compelling reason otherwise.<br>
<br>
I've copied aarch64, aarch64_be convention.<br></blockquote><div> </div><div>... why on earth did someone let aarch64 do that? That said, no one uses the underscore except apparently aarch64. powerpc, mips, arm, sparc, etc all avoid it.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
You mean you prefer without underscore?<br>
I think bpfle, bpfbe are less readable.<br></blockquote><div><br></div><div>Possibly. Also easier to remember if there's not an exception here.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Also there is no 'default' endianness.<br></blockquote><div><br></div><div>I had guessed native would be default endianness?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
So what do you have in mind specificialy?<br></blockquote><div><br></div><div>bpfle or bpfel for little endian etc to keep it matching every other target (except for aarch64 which, per usual, decided to deviate in a way that's not useful).</div><div><br></div><div>-eric</div></div></div>