[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