<div dir="ltr"><div>_OPENMP is defined in OpenMP standards which says every compiler supporting pre-processing should have it.<br></div><div>No clue which compiler doesn't support pre-processing nowadays.<br></div><div>At least Intel, IBM XL, GNU all have _OPENMP.<br><br></div><div>I first noticed the missing __OPENMP issue shows up on 5.3 and 5.4,6.x all have the same issue.<br>I reported it last year.<br><a href="http://qe-forge.org/pipermail/q-e-developers/2016-September/001355.html">http://qe-forge.org/pipermail/q-e-developers/2016-September/001355.html</a><br><br></div><div>Probably during the phase the BGQ is deployed in CINECA, that is maintained but not recently.<br></div><div><br>Best,<br></div><div>Ye<br></div><div><br></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">===================<br>
Ye Luo, Ph.D.<br>
Leadership Computing Facility<br>
Argonne National Laboratory</div></div></div>
<br><div class="gmail_quote">2017-03-17 15:52 GMT-05:00 Paolo Giannozzi <span dir="ltr"><<a href="mailto:p.giannozzi@gmail.com" target="_blank">p.giannozzi@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Ye<br>
<br>
the only reason to have __OPENMP with two underscores is that all<br>
preprocessing options in QE start with two underscores. If all<br>
OpenMP-aware compilers honor _OPENMP, I agree that it may be a good<br>
idea to drop one underscore and related "configure" stuff.<br>
<br>
The OpenMP-enabled code has been running for years on the BG in<br>
CINECA. I never noticed what you report.<br>
<br>
Paolo<br>
<div><div class="h5"><br>
On Fri, Mar 17, 2017 at 8:45 PM, Ye Luo <<a href="mailto:xw111luoye@gmail.com">xw111luoye@gmail.com</a>> wrote:<br>
> Ciao, Paolo,<br>
> Is there a particular reason to keep __OPENMP instead of _OPENMP (single<br>
> underscore) which is automatically added by the compiler when OpenMP is<br>
> requrested?<br>
> I'm asking this question because the __OPENMP flag is missing on BGQ on QE<br>
> 5.X to 6.X when --enable-openmp is added. Using the standard flag may help<br>
> reducing the maintenance pain of configure.<br>
><br>
> Ye<br>
> ===================<br>
> Ye Luo, Ph.D.<br>
> Leadership Computing Facility<br>
> Argonne National Laboratory<br>
><br>
</div></div>> ______________________________<wbr>_________________<br>
> Pw_forum mailing list<br>
> <a href="mailto:Pw_forum@pwscf.org">Pw_forum@pwscf.org</a><br>
> <a href="http://pwscf.org/mailman/listinfo/pw_forum" rel="noreferrer" target="_blank">http://pwscf.org/mailman/<wbr>listinfo/pw_forum</a><br>
<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
--<br>
Paolo Giannozzi, Dip. Scienze Matematiche Informatiche e Fisiche,<br>
Univ. Udine, via delle Scienze 208, 33100 Udine, Italy<br>
Phone <a href="tel:%2B39-0432-558216" value="+390432558216">+39-0432-558216</a>, fax <a href="tel:%2B39-0432-558222" value="+390432558222">+39-0432-558222</a><br>
______________________________<wbr>_________________<br>
Pw_forum mailing list<br>
<a href="mailto:Pw_forum@pwscf.org">Pw_forum@pwscf.org</a><br>
<a href="http://pwscf.org/mailman/listinfo/pw_forum" rel="noreferrer" target="_blank">http://pwscf.org/mailman/<wbr>listinfo/pw_forum</a><br>
</font></span></blockquote></div><br></div>