<div dir="ltr">Dear Andy, <div class="gmail_extra"><br><div class="gmail_quote">On Sun, Feb 14, 2016 at 1:27 PM, Andy Polyakov <span dir="ltr"><<a href="mailto:appro@openssl.org" target="_blank">appro@openssl.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> Does this patch work for you?<br>
<br>
</span>I'd vote against [catering this information in public header]. As it<br>
stands now -D[BL]_ENDIAN in OpenSSL config lines is actually just an<br>
optimization flag, and not as significant one. "Optimization flag" means<br>
that you can actually omit it, and it will still work, and "not as<br>
significant" means that not very much slower. Real endian dependencies<br>
are handled either by adhering to endian-neutral coding practices or so<br>
called constant conditions, such as if (endian.little). Advantage of<br>
constant condition is that both paths are parsed by compilers, so that<br>
at least code handling "less popular" endianness doesn't end up in<br>
majority's blind spot. There is no reason for why either of these<br>
techniques can't be exercised in off-tree code.<br></blockquote><div><br></div><div>The endianess information could be used in case of cross-compilation.</div><div> </div></div>-- <br><div class="gmail_signature">SY, Dmitry Belyavsky</div>
</div></div>