tianshilei1992 accepted this revision. tianshilei1992 added a comment. This revision is now accepted and ready to land.
Yeah, otherwise I suppose there will be some errors when compiling OpenMP program when there is no CUDA installed. ================ Comment at: clang/tools/nvptx-arch/NVPTXArch.cpp:89 int main() { + // Attempt to load the NVPTX driver runtime. ---------------- unrelated: I always prefer: ``` int main(int argc, char *argv[]) { return 0; } ``` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141861/new/ https://reviews.llvm.org/D141861 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits