[PATCH] D100514: [OpenMP] Added codegen for masked directive
Chi Chun Chen via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 16 10:23:26 PDT 2021
cchen added a comment.
In D100514#2693600 <https://reviews.llvm.org/D100514#2693600>, @jdoerfert wrote:
> Any reason we should not unconditionally use the OMPIRBuilder impl? (btw, many thanks for providing one!)
> We have an OMPIRBuilder always around in clang's codegen, so there is little reason not to use it if it is feature complete.
I'm fine using OMPIRBuilder as default. I was not set it as default since most of the clause/directive are still using Clang codegen as default (ex: `master`, `critical`, and `for` are now use Clang codegen as default).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D100514/new/
https://reviews.llvm.org/D100514
More information about the llvm-commits
mailing list