[PATCH] D47394: [OpenMP][Clang][NVPTX] Replace bundling with partial linking for the OpenMP NVPTX device offloading toolchain

Jonas Hahnfeld via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 1 07:18:35 PDT 2018


Hahnfeld added a comment.

In https://reviews.llvm.org/D47394#1118957, @gtbercea wrote:

>   I'm surprised you now disagree with this technique, when I first introduced you to this in an e-mail off list you agreed with it.


My words were `I agree this is the best solution for NVPTX.` In the same reply I asked how your proposal is supposed to work for other offloading targets which is now clear to require additional work, maybe even completely novel tools.
So now I disagree that it is the **right** solution for Clang because I think my proposal will cover all offloading targets. Please give me a bit time so that I can see if it works.


Repository:
  rC Clang

https://reviews.llvm.org/D47394





More information about the cfe-commits mailing list