[Openmp-dev] [RFC]Requiring C++14 for the OpenMP subproject

Wed Feb 12 06:22:28 PST 2020

On 02/12, Jim Cownie wrote:
> > For the record, that will not work for the device runtime. It has to be
> > compiled by Clang in the near future (maybe already now). This is not
> > only for the performance benefits we get when we link it into the
> > application and remove all the clutter we do actually not need but also
> > because we are working towards a native OpenMP device runtime. That is,
> > soon the OpenMP device runtime will contain as little foreign code
> > (CUDA/HIP/...) as possible in favor of native OpenMP code.
> 
> I am not sure what you are saying “will not work”.
> I am not really concerned with which compiler is used to build libomp. 
> (At one level it is clearly preferable to have code which is not compiler dependent, so can be compiled by any compiler,
> however if there are significant advantages in forcing the use of CLANG to compile the runtime that’s probably OK.)
> 
> What I don’t want is that libomp ends up with dependencies which make it hard to use with code compiled by other 
> compilers, at least when we’re only dealing with CPUs.
> As before, there are two cases there
> 1) Executable image compiled by GCC or LLVM, (shared) library compiled by the other compiler where both the executable and library use OpenMP.
> 2) All code compiled by GCC.
> 
> What I’d like to remain easy is the use of libomp in that second case. 
> 
> Once we bring in offloading, then everything probably breaks anyway, since I expect that we do things differently from GCC there and haven’t tried to create any shims.

I agree. My comment was *only* about the offloading part, libomptarget.
The shim layer is less useful here for performance reasons and due to
the fact that we should not commit to an interface. This is legitimate
given that we (should) expect libomptarget to be linked into the
application (as LLVM-IR).

I actually want us to link a single instance of the runtime per "type of
target region" soon. I mean you can then end up with two copies of
libomptarget.ll linked into the application one for SPMD and one for
non-SPMD. This is only necessary as long as we keep global state that
distinguishes the two modes, so getting rid of that would also a
possibility.

We can discuss this further at some point ;)

> > On 11 Feb 2020, at 17:25, Johannes Doerfert <jdoerfert at anl.gov> wrote:
> > 
> > On 02/11, Jim Cownie via Openmp-dev wrote:
> >>> On 2/11/20 4:56 AM, Jon Chesterfield via Openmp-dev wrote:
> >>>> 
> >>>>    Hmm, so you propose creating a dependency from the OpenMP runtime to
> >>>>    the LLVM libraries? Which classes would you want to use, probably some
> >>>>    of the ADTs? Does any of the other runtime libraries (libc++,
> >>>>    libc++abi, compiler-rt, sanitizers) already do this? IIRC
> >>>>    libc++abi and
> >>>>    LLVMSupport share some demangling code, but that is manually copied.
> >>>> 
> >>>> 
> >>>> I am unaware that each subproject reimplements code to avoid a 
> >>>> dependency on llvm. My first reaction is "surely not", but it's not 
> >>>> totally implausible. I will look into this.
> >>>> 
> >>>> ADT is the obvious contender for reuse. There will also be a lot of OS 
> >>>> wrappers somewhere. File IO, dlopen, probably memory allocators. 
> >>>> Essentially all the stuff a big cross platform C++ project grows over 
> >>>> time. In the case of anything hitting the filesystem, also 
> >>>> treacherously difficult to get right across platforms.
> >>>> 
> >>>> There's an open patch against libomptarget that uses dlopen with a 
> >>>> hardcoded path length. Opt loads shared libraries on windows afaik so 
> >>>> probably has a robust wrapper around dlopen, so I'd start there.
> >>> 
> >>> 
> >>> I would separate these two discussions. One motivation for the 
> >>> relicensing effort is to allow sharing code between runtime libraries 
> >>> and the core infrastructure. I expect that, as we move to full coverage 
> >>> on various parts of the relicensing, we'll have a large-scale discussion 
> >>> about how to implement this resuse across the project. However, we can 
> >>> certainly use C++14 language features before we worry about using LLVM's 
> >>> support library.
> >>> 
> >>>  -Hal
> >> 
> >> I agree wth Hal; there are two different issues here which have potentailly different implications :-
> >> Choice of base language level (C++11 or C++14)
> >> Moving code backwards and forwards between the runtime and the compiler, or adding additional runtime library requirements on users of the OpenMP runtime.
> >> 
> >> Changing the language dialect seems a small an uncontentious thing to
> >> do.  However adding additional dependencies seems more problematic. It
> >> is important that the runtime should remain usable with code compiled
> >> by GCC, for instance, even in the absence of other libraries. So not
> >> just that we support GCC’s OpenMP interfaces when we’re dealing with,
> >> say, an OpenMP program compiled by LLVM which links to a GCC compiled
> >> library that uses OpenMP, but also an all GCC case, where nothing but
> >> libomp has been compiled by LLVM.
> > 
> > For the record, that will not work for the device runtime. It has to be
> > compiled by Clang in the near future (maybe already now). This is not
> > only for the performance benefits we get when we link it into the
> > application and remove all the clutter we do actually not need but also
> > because we are working towards a native OpenMP device runtime. That is,
> > soon the OpenMP device runtime will contain as little foreign code
> > (CUDA/HIP/...) as possible in favor of native OpenMP code.
> > 
> >> This matters because libomp can provide useful performance
> >> improvements over libgomp at high core counts, and that is useful to
> >> some people.  It’s also a nice sales-pitch for LLVM in general :-
> >> “Look, you were able to use some of LLVM without even recompiling your
> >> GCC compiled code, and it improved the performance. Just think how
> >> good it could be if you moved to LLVM completely…” :-)
> > 
> > I agree and I do not want to change this feature but I believe for the
> > device runtime it is justified.
> > 
> > Cheers,
> >  Johannes
> > 
> > 
> >> -- Jim
> >> James Cownie <jcownie at gmail.com <mailto:jcownie at gmail.com>>
> >> Mob: +44 780 637 7146
> >> 
> >> 
> >> 
> >> 
> > 
> >> _______________________________________________
> >> Openmp-dev mailing list
> >> Openmp-dev at lists.llvm.org <mailto:Openmp-dev at lists.llvm.org>
> >> https://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-dev <https://lists.llvm.org/cgi-bin/mailman/listinfo/openmp-dev>
> > 
> > 
> > -- 
> > 
> > Johannes Doerfert
> > Researcher
> > 
> > Argonne National Laboratory
> > Lemont, IL 60439, USA
> > 
> > jdoerfert at anl.gov <mailto:jdoerfert at anl.gov>
> -- Jim
> James Cownie <jcownie at gmail.com>
> Mob: +44 780 637 7146
> 
> 
> 
> 

-- 

Johannes Doerfert
Researcher

Argonne National Laboratory
Lemont, IL 60439, USA

jdoerfert at anl.gov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-dev/attachments/20200212/255fbad9/attachment.sig>