[Openmp-commits] [PATCH] D105955: [NFC][OpenMP][Offloading] Replaced explicit parallel level computation with function `__kmpc_parallel_level`

[Johannes Doerfert via Phabricator via Openmp-commits]

jdoerfert added a comment.

In D105955#2880542 <https://reviews.llvm.org/D105955#2880542>, @tianshilei1992 wrote:

> In D105955#2876564 <https://reviews.llvm.org/D105955#2876564>, @JonChesterfield wrote:
>
>> The function interface is clang internal, let's change it to take void instead of a couple of ignored arguments. That will slightly simplify clang codegen and means the pass that looks for and replaces calls to it doesn't have to consider arguments (the latter may work out of the box)
>
> That's probably not working because as far as I can tell, we share some code generation with host code, but for host code, the two arguments are not ignored.

I don't know what you mean by not ignored. the function is device only.

Drop the print and the arguments, also consider adding a new api for other parallel level lookups, we might even just reuse this one.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D105955/new/

https://reviews.llvm.org/D105955