[Openmp-commits] [PATCH] D46842: [OpenMP][libomptarget] Make bitcode library building depend on clang and llvm-linker being available

Guansong Zhang via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Fri May 18 01:37:42 PDT 2018


guansong added a comment.

In https://reviews.llvm.org/D46842#1100723, @Hahnfeld wrote:

> In https://reviews.llvm.org/D46842#1099392, @guansong wrote:
>
> > Can we double check this LLVM IR backward compatible thing? My understanding is that it is not that good, at least for the major versions.
>
>
>
>
> In https://reviews.llvm.org/D46842#1099415, @guansong wrote:
>
> > I just want to comment this last point, my understanding is that it will not work even you try to inline older LLVM IR into a newer one.
>
>
> I did just that: I built libomptarget-nvptx-bc with Clang 3.9.1 (which is actually the only released version that you can currently build with) and inlined the result with LLVM/Clang trunk which will become 7.0. It worked without problems for all simple test cases I tried. Do you have a scenario that fails for you?


My understanding is that if things like the data layout is changed between versions, the old code will simply be wrong from the new version's perspective.


Repository:
  rOMP OpenMP

https://reviews.llvm.org/D46842





More information about the Openmp-commits mailing list