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

Joseph Huber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 2 09:34:17 PST 2021


jhuber6 added a comment.

In D114965#3167241 <https://reviews.llvm.org/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.

Well it's only for OpenMP with `-fopenmp`, but I guess it's fine to have that flag in the cc1 options even if offloading isn't used.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114965



More information about the cfe-commits mailing list