-
Notifications
You must be signed in to change notification settings - Fork 14.3k
[OpenMPIRBuilder][Clang][NFC] - Combine emitOffloadingArrays
and emitOffloadingArraysArgument
in OpenMPIRBuilder
#97088
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[OpenMPIRBuilder][Clang][NFC] - Combine emitOffloadingArrays
and emitOffloadingArraysArgument
in OpenMPIRBuilder
#97088
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One nit, otherwise LGTM.
…argetCallKernelLaunch in CGOpenMPRuntime.cpp
…elLaunch in CGOpenMPRuntime.cpp
…daloneCall in CGOpenMPRuntime.cpp
…t used anymore. Use emitOffloadingArraysAndArgs
…rgs that accept GenMapInfoCallBackTy
…apInfoCallBackTy in OMPIRBuilder.h
9d5c42b
to
e74f34b
Compare
✅ With the latest revision this PR passed the C/C++ code formatter. |
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/190/builds/2675 Here is the relevant piece of the build log for the reference:
|
This PR introduces a new interface in
OpenMPIRBuilder
that combines the creation of the so-called offloading pointer arrays and their subsequent preparation as arguments to the OpenMP runtime library. We then use this in Clang. This is intended to be used in the near future by other frontends such as Flang when lowering MLIR to LLVMIR.