jhuber6 added a comment. In D123460#3443925 <https://reviews.llvm.org/D123460#3443925>, @tianshilei1992 wrote:
> If we'd like it to be used by CUDA or other programming models, > `llvm/lib/Frontend/OpenMP/OMPIRBuilder.cpp` is not a good place. Other than > that, the patch looks good to me. I couldn't think of a better place to put this. Using it outside of OpenMP just requires initializing the OpenMP types so it's not a huge burden. I also want to change the names from `omp_offloading` so it's more generic but figured I wouldn't pollute the patch with those test changes. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D123460/new/ https://reviews.llvm.org/D123460 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
