[PATCH] D76184: [OpenMP][NFC] Remove the need to include `OpenMPClause.h`

Johannes Doerfert via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 23 15:48:55 PDT 2020


jdoerfert added a comment.

In D76184#1927159 <https://reviews.llvm.org/D76184#1927159>, @rnk wrote:

> I patched this in with it's dependency, and to compile just Attr.h by itself, it reduces the time from ~2.420s to ~1.920s, so it is worth pushing this through. :)
>
> However, I noticed that there are still some targets broken by the removal of the SmallSet.h include. I'm happy to push that through if you want to hand it off.


I was about to push all three changes but I wans't aware of broken targets. Thx for telling me. I would not mind you talking over this patch. I'll merged the other two.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D76184





More information about the cfe-commits mailing list