Skip to content

Commit bc5ce2c

Browse files
committed
[Comgr] Clean up HIP options
HIP by default should be compiled with -nogpuinc to not rely on unavailable clang headers Also we can replace --cuda-device-only with the synonymous --offload-device-only Finally, we can remove the hard-coded std=c++11, and instead rely on the clang default (currently c++17) Change-Id: I68ba0226013dad6708862987eb66a4e7487cb23b
1 parent 3cfd286 commit bc5ce2c

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

amd/comgr/src/comgr-compiler.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1056,8 +1056,8 @@ amd_comgr_status_t AMDGPUCompiler::addCompilationFlags() {
10561056
break;
10571057
case AMD_COMGR_LANGUAGE_HIP:
10581058
Args.push_back("hip");
1059-
Args.push_back("-std=c++11");
1060-
Args.push_back("--cuda-device-only");
1059+
Args.push_back("-nogpuinc");
1060+
Args.push_back("--offload-device-only");
10611061
Args.push_back("-isystem");
10621062
Args.push_back(ROCMIncludePath.c_str());
10631063
Args.push_back("-isystem");

0 commit comments

Comments
 (0)