[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