[PATCH] D109175: [openmp] Add clang cc1 option -fopenmp-skip-deferred-diags

Thu Sep 2 11:45:18 PDT 2021

weiwang added a comment.

In D109175#2980446 <https://reviews.llvm.org/D109175#2980446>, @lebedev.ri wrote:

> In D109175#2980333 <https://reviews.llvm.org/D109175#2980333>, @weiwang wrote:
>
>> Our internal codebase never uses the target directive. Once the deferred diags is bypassed, we observed 18% e2e build time improvement.
>
> Is that with `-fopenmp` or without?
> That seems, kinda a lot more than i would have expected,
> perhaps there are some other ways to reduce the overhead other than this approach?

This is with -fopenmp and no other omp related flags. I'd prefer a more generic way of fixing this, but right now this seems to be most direct way.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D109175/new/

https://reviews.llvm.org/D109175