[PATCH] D109885: [MLIR][[amdgpu-arch]][OpenMP] Remove direct dependency on /opt/rocm

Jon Chesterfield via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 15 06:54:37 PST 2021


JonChesterfield added a comment.

Diff was created without context unfortunately, scrolling through the cmake locally suggests variables have names of the form `LIBOMPTARGET_AMDGPU_FOO` though the namespacing isn't as consistent as it might be.

Something like:

  if (not defined (LIBOMPTARGET_AMDGPU_HSA_SEARCH_PATH))
  set(LIBOMPTARGET_AMDGPU_HSA_SEARCH_PATH "/opt/rocm")
  endif()

where I'm not confident of the cmake syntax for 'is this thing defined' or 'assign', looks like it might be `if (NOT LIBOMPTARGET_AMDGPU_HSA_SEARCH_PATH)` and `set`.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109885/new/

https://reviews.llvm.org/D109885



More information about the cfe-commits mailing list