jdoerfert added inline comments.
================ Comment at: lib/Driver/ToolChains/Clang.cpp:1159 + getToolChain().getTriple().isNVPTX()) + getToolChain().AddMathDeviceFunctions(Args, CmdArgs); + ---------------- Here is another "NVPTX" specialization that I don't think we need. At least with more targets we need to relax this condition. ================ Comment at: lib/Headers/__clang_openmp_math.h:13 + +#ifdef __NVPTX__ +#pragma omp declare target ---------------- Why is this NVPTX specific (again)? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60907/new/ https://reviews.llvm.org/D60907 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
