[PATCH] D104904: [OpenMP][AMDGCN] Initial math headers support

[Jon Chesterfield via Phabricator via cfe-commits]

JonChesterfield added a comment.

Good spot. I've been feeding the following to various toolchains:

  // permute
  //#include <math.h>
  //#include <cmath>
  //#include <complex.h>
  
  #ifndef _OPENMP
  #error "OpenMP should be defined
  #endif
  
  #ifdef __CUDA__
  #error "Cuda should not be defined"
  #endif
  
  #ifdef __HIP__
  #error "HIP should not be defined"
  #endif
  
  int main() {}

Currently, trunk passes that with any of the headers uncommented. Rocm is ok 
for math and cmath, but defines CUDA for complex.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104904

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits