[PATCH] D114965: [OpenMP] Remove the new runtime default for AMDGPU

2021-12-02 Thread Joseph Huber via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG96ff74a0d598: [OpenMP] Remove the new runtime default for AMDGPU (authored by jhuber6). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST AC

[PATCH] D114965: [OpenMP] Remove the new runtime default for AMDGPU

2021-12-02 Thread Joseph Huber via Phabricator via cfe-commits
jhuber6 added a comment. In D114965#3167241 , @jhuber6 wrote: > I feel like I should make these options only enabled if we're in an OpenMP > device. I just realized that this will probably enable it for anything that > goes through here, even non-OpenMP

[PATCH] D114965: [OpenMP] Remove the new runtime default for AMDGPU

2021-12-02 Thread Joseph Huber via Phabricator via cfe-commits
jhuber6 added a comment. I feel like I should make these options only enabled if we're in an OpenMP device. I just realized that this will probably enable it for anything that goes through here, even non-OpenMP. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews

[PATCH] D114965: [OpenMP] Remove the new runtime default for AMDGPU

2021-12-02 Thread Joseph Huber via Phabricator via cfe-commits
jhuber6 created this revision. jhuber6 added reviewers: jdoerfert, JonChesterfield, tianshilei1992, gregrodgers, ronlieb. Herald added subscribers: dang, kerbowa, guansong, t-tye, tpr, dstuttard, yaxunl, nhaehnle, jvesely, kzhuravl. jhuber6 requested review of this revision. Herald added subscrib