Skip to content

Commit

Permalink
[Libomptarget] Disable AMDGPU complex math test after recent patch
Browse files Browse the repository at this point in the history
Summary:
The recent patch added `-nogpulib` to make these tests only pick up what
was intentionally put into them. This had the effect of removing the
dependency on the ROCm device libs which are needed for math. This test
disables the complex math test, which is the only one that needed it,
for the time being. In the future we will implement these and provide it
via the GPU `libm` and pass it in the same way as the GPU `libc`.
  • Loading branch information
jhuber6 committed Oct 4, 2023
1 parent b6f2919 commit 75e6480
Showing 1 changed file with 4 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,10 @@
// RUN: %libomptarget-compilexx-generic -O3 -ffast-math && \
// RUN: %libomptarget-run-generic

// FIXME: This fails to link due to missing math symbols. We should provide the
// needed math functions in the GPU `libm` and require the GPU C library.
// UNSUPPORTED: amdgcn-amd-amdhsa

#include <cassert>
#include <complex>
#include <iostream>
Expand Down

0 comments on commit 75e6480

Please sign in to comment.