<div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Tue, Jun 23, 2015 at 11:37 AM Joerg Sonnenberger <<a href="mailto:joerg@britannica.bec.de">joerg@britannica.bec.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Jun 23, 2015 at 04:30:32PM +0000, Jonathan Peyton wrote:<br>
> 1) Renaming internal variables/macros/functions – everything has<br>
> libomp_ or LIBOMP_ prefixed to it unless maybe if it is a local<br>
> variable/parameter inside a function/macro.<br>
<br>
Which reminds me, why do we need or even want something like<br>
LIBOMP_ARCH and co? It is somewhat annoying that configuration flags<br>
exists for something that can be perfectly well handled in the code.<br>
Predefined platform macros exist for a reason.<br></blockquote><div><br></div><div>I very strongly agree. I think moving all of the arch detection to be based on predefined macros inside the source code would be a big improvement.</div><div><br></div><div>However, I'm OK if that comes in a follow-up patch. I'd like to make sure we make progress here.</div></div></div>