[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