oToToT added inline comments.
================ Comment at: clang/lib/Frontend/CompilerInstance.cpp:109 + // other side of CUDA/OpenMP/SYCL compilation. + if (!hasAuxTarget() && + (getLangOpts().CUDA || getLangOpts().OpenMPIsDevice || ---------------- kadircet wrote: > can we just do `!getAuxTarget()` and not introduce `hasAuxTarget()` ? i don't > think the new api in the interface provides much value. After some thinking, I’m wondering might it be more flexible if we add an option to let the user determine whether overwrite AuxTarget by themselves? Like changing the prototype to `bool CompilerInstance::createTarget(bool OverwriteAuxTarget = true);` and simply check `if (OverwriteAuxTarget && ...` here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100024/new/ https://reviews.llvm.org/D100024 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
