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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to