JonChesterfield added a comment.

Dropping PATHS /opt/rocm from the openmp parts will break people using an 
existing rocm install with trunk llvm. I think it would be reasonable to look 
at whatever ROCM_PATH is set to instead of assuming /opt/rocm. Is that a 
variable we can pass to find_package?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109885

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D109885: [M... DineshKumar Bhaskaran via Phabricator via cfe-commits
    • [PATCH] D109885... Jon Chesterfield via Phabricator via cfe-commits
    • [PATCH] D109885... Ye Luo via Phabricator via cfe-commits

Reply via email to