[PATCH] D121118: [flang] Add OpenMP flag to bbc
Shraiysh via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Mar 7 08:50:07 PST 2022
shraiysh added a comment.
In D121118#3364061 <https://reviews.llvm.org/D121118#3364061>, @clementval wrote:
> In D121118#3364024 <https://reviews.llvm.org/D121118#3364024>, @awarzynski wrote:
>
>> I'm not sure what the long-term direction is here, but AFAIK this change could be avoided and `flang-new` could be used for OpenMP tests instead.
>
> As long as `bbc` is in tree and it is used I think it makes sense to have the flag.
+1 on this. As long as we are keeping `bbc` in the tree, then I think it makes sense to have it. (Long term - maybe not - does `flang-new` use `bbc` internally or is there some code duplication?)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121118/new/
https://reviews.llvm.org/D121118
More information about the llvm-commits
mailing list