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

Jon Chesterfield via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jun 30 10:04:35 PDT 2021


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



More information about the cfe-commits mailing list