Complex math functions not linking on nvptx

[pdhaliwal-amd]

Bugzilla Link	51639
Resolution	FIXED
Resolved on	Sep 01, 2021 17:40
Version	trunk
OS	All
Blocks	#50580
CC	@jdoerfert,@jdoerfert,@tstellar
Fixed by commit(s)	2930c83 73c36a9

Extended Description

I have been trying to use complex math functions in the openmp target
region. I am able to use basic complex arithmetic but math functions
like abs, sin do not work. I get a linker undefined error when I use
these methods. I noticed that these methods are defined in
openmp_wrappers/complex_cmath.h in the variant region in
https://github.com/llvm/llvm-project/blob/main/clang/lib/Headers/openmp_wrappers/complex#L47.
Am I missing something?

More details,
command line: clang++ test.cpp -fopenmp -fopenmp-targets=nvptx64-nvidia-cuda
test.cpp:
#include
#include
#include

int main() {
std::complex x, a, b;
#pragma omp target map(tofrom: x)
{
x = std::exp(a * b);
}

std::cout << x << std::endl;
return 0;
}

linker error: nvlink error : Undefined reference to 'cexp' in
'/tmp/test-a02307.cubin'
CUDA details:
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2020 NVIDIA Corporation
Built on Tue_Sep_15_19:10:02_PDT_2020
Cuda compilation tools, release 11.1, V11.1.74
Build cuda_11.1.TC455_06.29069683_0

[jdoerfert]

Fixed by 5877356db6966409f4f9f2cdbcf3a07

@​Tom can we back port this too.

[tstellar]

Fixed by 5877356db6966409f4f9f2cdbcf3a07

@​Tom can we back port this too.

Is the fix 5877356db6966409f4f9f2cdbcf3a07 or 2930c83 ?

[jdoerfert]

Fixed by 5877356db6966409f4f9f2cdbcf3a07

@​Tom can we back port this too.

Is the fix 5877356db6966409f4f9f2cdbcf3a07 or
2930c83 ?

5877... is a suffix of the actual 2930c... git hash.
I managed to loose the first few digits somehow.
The fix is: 2930c83

[tstellar]

Merged: 73c36a9