[PATCH] D47849: [OpenMP][Clang][NVPTX] Enable math functions called in an OpenMP NVPTX target device region to be resolved as device-native function calls

[Gheorghe-Teodor Bercea via Phabricator via cfe-commits]

gtbercea added a comment.

In https://reviews.llvm.org/D47849#1192368, @Hahnfeld wrote:

> In https://reviews.llvm.org/D47849#1192321, @gtbercea wrote:
>
> > > IIRC you started to work on this to fix the problem with inline assembly (see https://reviews.llvm.org/D47849#1125019). AFAICS this patch fixes declarations of math functions but you still cannot include `math.h` which most "correct" codes do.
> >
> > I'm not sure what you mean by this. This patch enables me to include math.h.
>
>
> `math.c`:
>
>   #include <math.h>
>
>
> executed commands:
>
>    $ clang -fopenmp -fopenmp-targets=nvptx64-nvidia-cuda -c math.c -O2
>   In file included from math.c:1:
>   In file included from /usr/include/math.h:413:
>   /usr/include/bits/mathinline.h:131:43: error: invalid input constraint 'x' in asm
>     __asm ("pmovmskb %1, %0" : "=r" (__m) : "x" (__x));
>                                             ^
>   /usr/include/bits/mathinline.h:143:43: error: invalid input constraint 'x' in asm
>     __asm ("pmovmskb %1, %0" : "=r" (__m) : "x" (__x));
>                                             ^
>   2 errors generated.
>


I do not get that error.


Repository:
  rC Clang

https://reviews.llvm.org/D47849