[Comgr] Add missing clangOptions to Comgr link libraries

[jmmartinez]

Fixes error when compiling with -DBUILD_SHARED_LIBS=ON:

  /usr/bin/ld:
  tools/comgr/CMakeFiles/amd_comgr.dir/src/comgr-compiler.cpp.o: in
  function
  `COMGR::AMDGPUCompiler::executeInProcessDriver(llvm::ArrayRef<char
  const*>)':
  amd/comgr/src/comgr-compiler.cpp:739:
  undefined reference to `clang::getDriverOptTable()'
  collect2: error: ld returned 1 exit status
  ninja: build stopped: subcommand failed.

Related to llvm#167374

[z1-cciauto]

PSDB Link: https://compiler-ci.amd.com/job/compiler-psdb-amd-staging/2871

[lamb-j]

Thanks for catching this. We should try to make sure this patch makes it into the next promotion to TheRock, as I believe they're using -DBUILD_SHARED_LIBS=ON

[jmmartinez]

Thanks for catching this. We should try to make sure this patch makes it into the next promotion to TheRock, as I believe they're using -DBUILD_SHARED_LIBS=ON

Do you know how do we do that ? (e.g. is there a CP list)

[lamb-j]

Good question. Right now TheRock hashes are coming from our amd-mainline branch, so saftest bet would be to CP this one to mainline. Will this break anything if we don't also pull in llvm#167374?

[jmmartinez]

Good question. Right now TheRock hashes are coming from our amd-mainline branch, so saftest bet would be to CP this one to mainline. Will this break anything if we don't also pull in llvm#167374?

Yes, it'd be broken since clangOptions doesn't exist before that PR.