[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