[PATCH] D138391: clang/HIP: Add new header test for math IR gen
    Jon Chesterfield via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Tue Nov 22 01:25:19 PST 2022
    
    
  
JonChesterfield accepted this revision.
JonChesterfield added a comment.
This revision is now accepted and ready to land.
Good call adding this test, thanks.
I think some of the skipping is glibc quirk related and some was to avoid clobbering the user's namespace for openmp. The math header defines various functions at global scope which can collide with user symbols. In practice that may not be a big deal.
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138391/new/
https://reviews.llvm.org/D138391
    
    
More information about the cfe-commits
mailing list