[PATCH] D9888: [OPENMP] Driver support for OpenMP offloading
Jonas Hahnfeld via cfe-commits
cfe-commits at lists.llvm.org
Thu Oct 8 00:23:50 PDT 2015
Hahnfeld added a comment.
In http://reviews.llvm.org/D9888#262389, @sfantao wrote:
> [...]
>
> I assume you were trying this using the diff in http://reviews.llvm.org/D12614. There was an inconsistency in the names of the ELF sections and symbols defined by the linker script in these two patches. This is now fixed.
>
> Note that if you are using the libomptarget library from clang-omp, you need to replace in the code `.openmptgt_host_entries` by `.omp_offloading.entries`. I changed the names so that all of them are consistent with what is already in place for other OpenMP directives.
>
> I also changed the files generation so that different files are used even if target and host have the same triple.
>
> Please, let me know if it still does not work for you.
Thanks for your help, a small test program now seems to work!
> [...]
>
> I didn't implement the triples logic for the nvptx targets yet. I'll port that from clang-omp once we have the basic functionality working upstream.
Ok, I'll wait then. Thanks for your work and finally upstreaming this!
Jonas
http://reviews.llvm.org/D9888
More information about the cfe-commits
mailing list