[PATCH] D99402: [AMDGPU][OpenMP] Add /include to the search path

pdhaliwal added a comment. The general problem seems bit more involved. I am not that familiar with how other architectures/systems handle the library/include path. Simplest solution that I can propose right now is to generalise my revision to other architectures for header lookup and similarl

[PATCH] D99402: [AMDGPU][OpenMP] Add /include to the search path

JonChesterfield added a comment. The general problem looks harder but important to fix. Finding the right headers but the wrong shared library is bad, and iirc we currently have to use LD_LIBRARY_PATH to bodge the latter which is not a good UX. Repository: rG LLVM Github Monorepo CHANGES SI

[PATCH] D99402: [AMDGPU][OpenMP] Add /include to the search path

pdhaliwal added a comment. I was not aware that it was a general problem. I will check if I could get a general solution. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99402/new/ https://reviews.llvm.org/D99402 ___

[PATCH] D99402: [AMDGPU][OpenMP] Add /include to the search path

lebedev.ri added a comment. Any chance you might be interested in fixing the more general problem, benefiting every clang openmp user, not just amdgpu? D55725 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99402/new

[PATCH] D99402: [AMDGPU][OpenMP] Add /include to the search path

pdhaliwal created this revision. pdhaliwal added reviewers: ronlieb, JonChesterfield, jdoerfert. Herald added subscribers: kerbowa, guansong, t-tye, tpr, dstuttard, yaxunl, nhaehnle, jvesely, kzhuravl. pdhaliwal requested review of this revision. Herald added subscribers: cfe-commits, sstefan1, wd