[Openmp-commits] [PATCH] D158546: [openmp] Fix flang detection for offloading test

[Ethan Luis McDonough via Phabricator via Openmp-commits]

elmcdonough added a comment.

In D158546#4610557 <https://reviews.llvm.org/D158546#4610557>, @jsjodin wrote:

> This change works for me, so the test is run and fails with:
>
>   ld.lld: error: undefined symbol: increment_at
>   >>>> referenced by /tmp/lit-tmp-hsj2_6gq/basic_array.c.tmp.amdgcn-amd-amdhsa.gfx90a-465f86.o:(__omp_offloading_802_ae2379e_increment_array_l17)
>   >>>> referenced by /tmp/lit-tmp-hsj2_6gq/basic_array.c.tmp.amdgcn-amd-amdhsa.gfx90a-465f86.o:(__omp_offloading_802_ae2379e_increment_array_l17)
>   clang: error: ld.lld command failed with exit code 1 (use -v to see invocation)
>   /home/jsjodin/git/trunk/llvm-project/build/./bin/clang-linker-wrapper: error: 'clang' failed
>   clang: error: linker command failed with exit code 1 (use -v to see invocation)
>   
>   error: command failed with exit status: 1

Yeah, that test fails at the moment.  Right now it seems that `increment_at` isn't registered as a symbol in the LLVM bitcode for the embedded object.  I can confirm that this used to work, but I'm still trying to figure out why it doesn't anymore.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D158546/new/

https://reviews.llvm.org/D158546