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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to