<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p><br>
</p>
<div class="moz-cite-prefix">On 10/02/2017 07:38 PM, Alexey Bataev
wrote:<br>
</div>
<blockquote
cite="mid:AM5PR0902MB1923231045A5258FC5D610C085720@AM5PR0902MB1923.eurprd09.prod.outlook.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
No, there is no such page. We parse everything from 4.5, but have
very limited support in codegen for target-specific directives,
especially combined one. Moreover, some of them are not
implemented at all and we may produce incorrect code. I can try to
revisit these "badly" supported constructs and provide some basic
codegen to produce working code at least, though without actual
offloading. But not sure that I will be able to do it quickly. <br>
</blockquote>
<br>
I think that it would be useful to have a status page for OpenMP.
Something like <a class="moz-txt-link-freetext" href="https://clang.llvm.org/cxx_status.html">https://clang.llvm.org/cxx_status.html</a>. If you could
make something like that (you're probably the best person to do it),
that would be great.<br>
<br>
We might get other people to help contribute to some of the missing
areas - I suspect that this is easier if we can enumerate them.<br>
<br>
Thanks again,<br>
Hal<br>
<br>
<blockquote
cite="mid:AM5PR0902MB1923231045A5258FC5D610C085720@AM5PR0902MB1923.eurprd09.prod.outlook.com"
type="cite">
<br>
<div id="AppleMailSignature">Best regards,
<div>Alexey Bataev</div>
</div>
<div><br>
2 окт. 2017 г., в 20:22, Hal Finkel <<a
moz-do-not-send="true" href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>>
написал(а):<br>
<br>
</div>
<blockquote type="cite">
<div>
<p><br>
</p>
<div class="moz-cite-prefix">On 10/02/2017 07:08 PM, Alexey
Bataev wrote:<br>
</div>
<blockquote
cite="mid:AM5PR0902MB19236875EAE7FE2C0E7EEE4985720@AM5PR0902MB1923.eurprd09.prod.outlook.com"
type="cite">
Hi Hal,
<div>As soon as we get the support for 4.5, including
offloading. Otherwise there always are going to be some
people blaming the compiler for not supporting 4.5 in
full. Will try to support it ASAP. </div>
<div>Meanwhile, you can use -fopenmp-version=45 option to
force to 4.5 <br>
</div>
</blockquote>
<br>
Thanks!<br>
<br>
Do we have a status page anywhere that shows where we stand on
the various features? Are there still features that we don't
parse, or where we don't generate something that
conservatively correct (e.g., as with "declare simd")?<br>
<br>
-Hal<br>
<br>
<blockquote
cite="mid:AM5PR0902MB19236875EAE7FE2C0E7EEE4985720@AM5PR0902MB1923.eurprd09.prod.outlook.com"
type="cite">
<div><br>
<div id="AppleMailSignature">Best regards,
<div>Alexey Bataev</div>
</div>
<div><br>
2 окт. 2017 г., в 19:53, Hal Finkel <<a
moz-do-not-send="true" href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>>
написал(а):<br>
<br>
</div>
<blockquote type="cite">
<div><span>Hi, Alexey,</span><br>
<span></span><br>
<span>At what point can we switch, by default, to
reporting a version for _OPENMP corresponding to
4.x? We're missing out on some OpenMP simd
directives because the source code guards them with
'#if _OPENMP >= 201307' or similar.</span><br>
<span></span><br>
<span>Thanks again,</span><br>
<span>Hal</span><br>
<span></span><br>
<span>On 05/26/2016 11:13 PM, Alexey Bataev via
cfe-commits wrote:</span><br>
<blockquote type="cite"><span>Author: abataev</span><br>
</blockquote>
<blockquote type="cite"><span>Date: Thu May 26
23:13:39 2016</span><br>
</blockquote>
<blockquote type="cite"><span>New Revision: 270962</span><br>
</blockquote>
<blockquote type="cite"><span></span><br>
</blockquote>
<blockquote type="cite"><span>URL: <a
moz-do-not-send="true"
href="http://llvm.org/viewvc/llvm-project?rev=270962&view=rev">
http://llvm.org/viewvc/llvm-project?rev=270962&view=rev</a></span><br>
</blockquote>
<blockquote type="cite"><span>Log:</span><br>
</blockquote>
<blockquote type="cite"><span>[OPENMP] Fixed
processing of '-fopenmp-version=' option and test.</span><br>
</blockquote>
<blockquote type="cite"><span></span><br>
</blockquote>
<blockquote type="cite"><span>Modified:</span><br>
</blockquote>
<blockquote type="cite"><span> cfe/trunk/lib/Driver/Tools.cpp</span><br>
</blockquote>
<blockquote type="cite"><span> cfe/trunk/lib/Frontend/CompilerInvocation.cpp</span><br>
</blockquote>
<blockquote type="cite"><span> cfe/trunk/lib/Frontend/InitPreprocessor.cpp</span><br>
</blockquote>
<blockquote type="cite"><span> cfe/trunk/test/OpenMP/driver.c</span><br>
</blockquote>
<blockquote type="cite"><span></span><br>
</blockquote>
<blockquote type="cite"><span>Modified:
cfe/trunk/lib/Driver/Tools.cpp</span><br>
</blockquote>
<blockquote type="cite"><span>URL: <a
moz-do-not-send="true"
href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=270962&r1=270961&r2=270962&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Driver/Tools.cpp?rev=270962&r1=270961&r2=270962&view=diff</a></span><br>
</blockquote>
<blockquote type="cite"><span>==============================================================================</span><br>
</blockquote>
<blockquote type="cite"><span>---
cfe/trunk/lib/Driver/Tools.cpp (original)</span><br>
</blockquote>
<blockquote type="cite"><span>+++
cfe/trunk/lib/Driver/Tools.cpp Thu May 26 23:13:39
2016</span><br>
</blockquote>
<blockquote type="cite"><span>@@ -4864,7 +4864,6 @@
void Clang::ConstructJob(Compilation &C,</span><br>
</blockquote>
<blockquote type="cite"><span> // Forward flags for
OpenMP</span><br>
</blockquote>
<blockquote type="cite"><span> if
(Args.hasFlag(options::OPT_fopenmp,
options::OPT_fopenmp_EQ,</span><br>
</blockquote>
<blockquote type="cite"><span> options::OPT_fno_openmp,
false)) {</span><br>
</blockquote>
<blockquote type="cite"><span>-
Args.AddAllArgs(CmdArgs,
options::OPT_fopenmp_version_EQ);</span><br>
</blockquote>
<blockquote type="cite"><span> switch
(getOpenMPRuntime(getToolChain(), Args)) {</span><br>
</blockquote>
<blockquote type="cite"><span> case OMPRT_OMP:</span><br>
</blockquote>
<blockquote type="cite"><span> case OMPRT_IOMP5:</span><br>
</blockquote>
<blockquote type="cite"><span>@@ -4877,6 +4876,7 @@
void Clang::ConstructJob(Compilation &C,</span><br>
</blockquote>
<blockquote type="cite"><span> if
(!Args.hasFlag(options::OPT_fopenmp_use_tls,</span><br>
</blockquote>
<blockquote type="cite"><span> options::OPT_fnoopenmp_use_tls,
/*Default=*/true))</span><br>
</blockquote>
<blockquote type="cite"><span> CmdArgs.push_back("-fnoopenmp-use-tls");</span><br>
</blockquote>
<blockquote type="cite"><span>+
Args.AddAllArgs(CmdArgs,
options::OPT_fopenmp_version_EQ);</span><br>
</blockquote>
<blockquote type="cite"><span> break;</span><br>
</blockquote>
<blockquote type="cite"><span> default:</span><br>
</blockquote>
<blockquote type="cite"><span> // By default, if
Clang doesn't know how to generate useful OpenMP
code</span><br>
</blockquote>
<blockquote type="cite"><span></span><br>
</blockquote>
<blockquote type="cite"><span>Modified:
cfe/trunk/lib/Frontend/CompilerInvocation.cpp</span><br>
</blockquote>
<blockquote type="cite"><span>URL: <a
moz-do-not-send="true"
href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=270962&r1=270961&r2=270962&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/CompilerInvocation.cpp?rev=270962&r1=270961&r2=270962&view=diff</a></span><br>
</blockquote>
<blockquote type="cite"><span>==============================================================================</span><br>
</blockquote>
<blockquote type="cite"><span>---
cfe/trunk/lib/Frontend/CompilerInvocation.cpp
(original)</span><br>
</blockquote>
<blockquote type="cite"><span>+++
cfe/trunk/lib/Frontend/CompilerInvocation.cpp Thu
May 26 23:13:39 2016</span><br>
</blockquote>
<blockquote type="cite"><span>@@ -1954,15 +1954,16 @@
static void ParseLangArgs(LangOptions &O</span><br>
</blockquote>
<blockquote type="cite"><span> }</span><br>
</blockquote>
<blockquote type="cite"><span> // Check if
-fopenmp is specified.</span><br>
</blockquote>
<blockquote type="cite"><span>- Opts.OpenMP =
Args.hasArg(options::OPT_fopenmp);</span><br>
</blockquote>
<blockquote type="cite"><span>+ Opts.OpenMP =
Args.hasArg(options::OPT_fopenmp) ? 1 : 0;</span><br>
</blockquote>
<blockquote type="cite"><span> Opts.OpenMPUseTLS =</span><br>
</blockquote>
<blockquote type="cite"><span> Opts.OpenMP
&&
!Args.hasArg(options::OPT_fnoopenmp_use_tls);</span><br>
</blockquote>
<blockquote type="cite"><span> Opts.OpenMPIsDevice =</span><br>
</blockquote>
<blockquote type="cite"><span> Opts.OpenMP
&&
Args.hasArg(options::OPT_fopenmp_is_device);</span><br>
</blockquote>
<blockquote type="cite"><span> if (Opts.OpenMP) {</span><br>
</blockquote>
<blockquote type="cite"><span>- if (int Version =
getLastArgIntValue(Args, OPT_fopenmp_version_EQ,</span><br>
</blockquote>
<blockquote type="cite"><span>-
Opts.OpenMP,
Diags))</span><br>
</blockquote>
<blockquote type="cite"><span>+ int Version =</span><br>
</blockquote>
<blockquote type="cite"><span>+
getLastArgIntValue(Args,
OPT_fopenmp_version_EQ, Opts.OpenMP, Diags);</span><br>
</blockquote>
<blockquote type="cite"><span>+ if (Version != 0)</span><br>
</blockquote>
<blockquote type="cite"><span> Opts.OpenMP =
Version;</span><br>
</blockquote>
<blockquote type="cite"><span> // Provide
diagnostic when a given target is not expected to
be an OpenMP</span><br>
</blockquote>
<blockquote type="cite"><span> // device or host.</span><br>
</blockquote>
<blockquote type="cite"><span></span><br>
</blockquote>
<blockquote type="cite"><span>Modified:
cfe/trunk/lib/Frontend/InitPreprocessor.cpp</span><br>
</blockquote>
<blockquote type="cite"><span>URL: <a
moz-do-not-send="true"
href="http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/InitPreprocessor.cpp?rev=270962&r1=270961&r2=270962&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Frontend/InitPreprocessor.cpp?rev=270962&r1=270961&r2=270962&view=diff</a></span><br>
</blockquote>
<blockquote type="cite"><span>==============================================================================</span><br>
</blockquote>
<blockquote type="cite"><span>---
cfe/trunk/lib/Frontend/InitPreprocessor.cpp
(original)</span><br>
</blockquote>
<blockquote type="cite"><span>+++
cfe/trunk/lib/Frontend/InitPreprocessor.cpp Thu
May 26 23:13:39 2016</span><br>
</blockquote>
<blockquote type="cite"><span>@@ -922,24 +922,24 @@
static void InitializePredefinedMacros(c</span><br>
</blockquote>
<blockquote type="cite"><span> }</span><br>
</blockquote>
<blockquote type="cite"><span> // OpenMP
definition</span><br>
</blockquote>
<blockquote type="cite"><span>- if (LangOpts.OpenMP)
{</span><br>
</blockquote>
<blockquote type="cite"><span>- // OpenMP 2.2:</span><br>
</blockquote>
<blockquote type="cite"><span>- // In
implementations that support a preprocessor, the
_OPENMP</span><br>
</blockquote>
<blockquote type="cite"><span>- // macro name is
defined to have the decimal value yyyymm where</span><br>
</blockquote>
<blockquote type="cite"><span>- // yyyy and mm
are the year and the month designations of the</span><br>
</blockquote>
<blockquote type="cite"><span>- // version of the
OpenMP API that the implementation support.</span><br>
</blockquote>
<blockquote type="cite"><span>- switch
(LangOpts.OpenMP) {</span><br>
</blockquote>
<blockquote type="cite"><span>- case 40:</span><br>
</blockquote>
<blockquote type="cite"><span>-
Builder.defineMacro("_OPENMP", "201307");</span><br>
</blockquote>
<blockquote type="cite"><span>- break;</span><br>
</blockquote>
<blockquote type="cite"><span>- case 45:</span><br>
</blockquote>
<blockquote type="cite"><span>-
Builder.defineMacro("_OPENMP", "201511");</span><br>
</blockquote>
<blockquote type="cite"><span>- break;</span><br>
</blockquote>
<blockquote type="cite"><span>- default:</span><br>
</blockquote>
<blockquote type="cite"><span>- // Default
version is OpenMP 3.1</span><br>
</blockquote>
<blockquote type="cite"><span>-
Builder.defineMacro("_OPENMP", "201107");</span><br>
</blockquote>
<blockquote type="cite"><span>- break;</span><br>
</blockquote>
<blockquote type="cite"><span>- }</span><br>
</blockquote>
<blockquote type="cite"><span>+ // OpenMP 2.2:</span><br>
</blockquote>
<blockquote type="cite"><span>+ // In
implementations that support a preprocessor, the
_OPENMP</span><br>
</blockquote>
<blockquote type="cite"><span>+ // macro name is
defined to have the decimal value yyyymm where</span><br>
</blockquote>
<blockquote type="cite"><span>+ // yyyy and mm are
the year and the month designations of the</span><br>
</blockquote>
<blockquote type="cite"><span>+ // version of the
OpenMP API that the implementation support.</span><br>
</blockquote>
<blockquote type="cite"><span>+ switch
(LangOpts.OpenMP) {</span><br>
</blockquote>
<blockquote type="cite"><span>+ case 0:</span><br>
</blockquote>
<blockquote type="cite"><span>+ break;</span><br>
</blockquote>
<blockquote type="cite"><span>+ case 40:</span><br>
</blockquote>
<blockquote type="cite"><span>+
Builder.defineMacro("_OPENMP", "201307");</span><br>
</blockquote>
<blockquote type="cite"><span>+ break;</span><br>
</blockquote>
<blockquote type="cite"><span>+ case 45:</span><br>
</blockquote>
<blockquote type="cite"><span>+
Builder.defineMacro("_OPENMP", "201511");</span><br>
</blockquote>
<blockquote type="cite"><span>+ break;</span><br>
</blockquote>
<blockquote type="cite"><span>+ default:</span><br>
</blockquote>
<blockquote type="cite"><span>+ // Default version
is OpenMP 3.1</span><br>
</blockquote>
<blockquote type="cite"><span>+
Builder.defineMacro("_OPENMP", "201107");</span><br>
</blockquote>
<blockquote type="cite"><span>+ break;</span><br>
</blockquote>
<blockquote type="cite"><span> }</span><br>
</blockquote>
<blockquote type="cite"><span> // CUDA device path
compilaton</span><br>
</blockquote>
<blockquote type="cite"><span></span><br>
</blockquote>
<blockquote type="cite"><span>Modified:
cfe/trunk/test/OpenMP/driver.c</span><br>
</blockquote>
<blockquote type="cite"><span>URL: <a
moz-do-not-send="true"
href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/OpenMP/driver.c?rev=270962&r1=270961&r2=270962&view=diff">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/OpenMP/driver.c?rev=270962&r1=270961&r2=270962&view=diff</a></span><br>
</blockquote>
<blockquote type="cite"><span>==============================================================================</span><br>
</blockquote>
<blockquote type="cite"><span>---
cfe/trunk/test/OpenMP/driver.c (original)</span><br>
</blockquote>
<blockquote type="cite"><span>+++
cfe/trunk/test/OpenMP/driver.c Thu May 26 23:13:39
2016</span><br>
</blockquote>
<blockquote type="cite"><span>@@ -8,17 +8,17 @@</span><br>
</blockquote>
<blockquote type="cite"><span> // CHECK-NO-TLS: -cc1</span><br>
</blockquote>
<blockquote type="cite"><span> // CHECK-NO-TLS-SAME:
-fnoopenmp-use-tls</span><br>
</blockquote>
<blockquote type="cite"><span> //</span><br>
</blockquote>
<blockquote type="cite"><span>-// RUN: %clang %s -c -E
-dM -fopenmp | FileCheck
--check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>-// RUN: %clang %s -c -E
-dM -fopenmp -fopenmp-version=1 | FileCheck
--check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>-// RUN: %clang %s -c -E
-dM -fopenmp -fopenmp-version=0 | FileCheck
--check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>-// RUN: %clang %s -c -E
-dM -fopenmp -fopenmp-version=100 | FileCheck
--check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>-// RUN: %clang %s -c -E
-dM -fopenmp -fopenmp-version=31 | FileCheck
--check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>+// RUN: %clang %s -c -E
-dM -fopenmp=libomp | FileCheck
--check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>+// RUN: %clang %s -c -E
-dM -fopenmp=libomp -fopenmp-version=1 | FileCheck
--check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>+// RUN: %clang %s -c -E
-dM -fopenmp=libomp -fopenmp-version=0 | FileCheck
--check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>+// RUN: %clang %s -c -E
-dM -fopenmp=libomp -fopenmp-version=100 |
FileCheck --check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>+// RUN: %clang %s -c -E
-dM -fopenmp=libomp -fopenmp-version=31 |
FileCheck --check-prefix=CHECK-DEFAULT-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span> //
CHECK-DEFAULT-VERSION: #define _OPENMP 201107</span><br>
</blockquote>
<blockquote type="cite"><span> -// RUN: %clang %s -c
-E -dM -fopenmp -fopenmp-version=40 | FileCheck
--check-prefix=CHECK-40-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>+// RUN: %clang %s -c -E
-dM -fopenmp=libomp -fopenmp-version=40 |
FileCheck --check-prefix=CHECK-40-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span> // CHECK-40-VERSION:
#define _OPENMP 201307</span><br>
</blockquote>
<blockquote type="cite"><span> -// RUN: %clang %s -c
-E -dM -fopenmp -fopenmp-version=45 | FileCheck
--check-prefix=CHECK-45-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span>+// RUN: %clang %s -c -E
-dM -fopenmp=libomp -fopenmp-version=45 |
FileCheck --check-prefix=CHECK-45-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span> // CHECK-45-VERSION:
#define _OPENMP 201511</span><br>
</blockquote>
<blockquote type="cite"><span> // RUN: %clang %s -c
-E -dM -fopenmp-version=1 | FileCheck
--check-prefix=CHECK-VERSION %s</span><br>
</blockquote>
<blockquote type="cite"><span></span><br>
</blockquote>
<blockquote type="cite"><span></span><br>
</blockquote>
<blockquote type="cite"><span>_______________________________________________</span><br>
</blockquote>
<blockquote type="cite"><span>cfe-commits mailing list</span><br>
</blockquote>
<blockquote type="cite"><span><a
moz-do-not-send="true"
href="mailto:cfe-commits@lists.llvm.org">cfe-commits@lists.llvm.org</a></span><br>
</blockquote>
<blockquote type="cite"><span><a
moz-do-not-send="true"
href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a></span><br>
</blockquote>
<span></span><br>
<span>-- </span><br>
<span>Hal Finkel</span><br>
<span>Lead, Compiler Technology and Programming
Languages</span><br>
<span>Leadership Computing Facility</span><br>
<span>Argonne National Laboratory</span><br>
<span></span><br>
</div>
</blockquote>
</div>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
Hal Finkel
Lead, Compiler Technology and Programming Languages
Leadership Computing Facility
Argonne National Laboratory</pre>
</div>
</blockquote>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
Hal Finkel
Lead, Compiler Technology and Programming Languages
Leadership Computing Facility
Argonne National Laboratory</pre>
</body>
</html>