diff --git a/amd/comgr/docs/ReleaseNotes.md b/amd/comgr/docs/ReleaseNotes.md index 6c1bcb10dec9bb..0185f2a987b541 100644 --- a/amd/comgr/docs/ReleaseNotes.md +++ b/amd/comgr/docs/ReleaseNotes.md @@ -59,10 +59,7 @@ lld::elf::link in Comgr's linkWithLLD() - Added -x assembler option to assembly compilation. Before, if an assembly file did not end with a .s file extension, it was not handled properly by the Comgr ASSEMBLE\_SOURCE\_TO\_RELOCATABLE action. -- Added new -relink-builtin-bitcode-postop LLVM option to device library. This -fixes an issue with the \*COMPILE\_SOURCE\_WITH\_DEVICE\_LIBRARIES\_TO\_BC where -OpenCL applications that leveraged AMDGPUSimplifyLibCalls optimizations would -need to re-link bitcodes separately to avoid errors at runtime. + New APIs -------- diff --git a/amd/comgr/src/comgr-compiler.cpp b/amd/comgr/src/comgr-compiler.cpp index dbf660d3b2379a..81f4fae0c55a9b 100644 --- a/amd/comgr/src/comgr-compiler.cpp +++ b/amd/comgr/src/comgr-compiler.cpp @@ -1074,8 +1074,6 @@ amd_comgr_status_t AMDGPUCompiler::addDeviceLibraries() { return AMD_COMGR_STATUS_ERROR; } Args.push_back(Saver.save(Twine("--rocm-path=") + FakeRocmDir).data()); - Args.push_back("-mllvm"); - Args.push_back("-relink-builtin-bitcode-postop"); NoGpuLib = false; for (auto DeviceLib : getDeviceLibraries()) { diff --git a/amd/comgr/test/compile_source_with_device_libs_to_bc_test.c b/amd/comgr/test/compile_source_with_device_libs_to_bc_test.c index f092a539d21102..5966ab4c43cde1 100644 --- a/amd/comgr/test/compile_source_with_device_libs_to_bc_test.c +++ b/amd/comgr/test/compile_source_with_device_libs_to_bc_test.c @@ -48,7 +48,7 @@ int main(int argc, char *argv[]) { amd_comgr_action_info_t DataAction; amd_comgr_status_t Status; const char *CodeGenOptions[] = {"-mllvm", "-amdgpu-early-inline-all", - "-mcode-object-version=5", "-mllvm", "-amdgpu-prelink"}; + "-mcode-object-version=5"}; size_t CodeGenOptionsCount = sizeof(CodeGenOptions) / sizeof(CodeGenOptions[0]);