ashi1 added inline comments.
================ Comment at: clang/lib/Headers/__clang_hip_cmath.h:20 #include <limits> #include <type_traits> #include <utility> ---------------- we may not need to `#include` limits and/or type_traits if we replaced them with `__hip::is_integral` and `__hip::is_arithmetic`. ================ Comment at: clang/lib/Headers/__clang_hip_cmath.h:587 #endif -#endif +#endif // !defined(__HIPCC_RTC__) ---------------- I don't think this `#endif` is for `!defined(__HIPCC_RTC__)`. It should be for `_LIBCPP_BEGIN_NAMESPACE_STD`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100794/new/ https://reviews.llvm.org/D100794 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
