[llvm-commits] [PATCH] Move the implementation of LoopInfo into LoopInfoImpl.h.

Chandler Carruth chandlerc at google.com
Thu May 24 22:15:22 PDT 2012


On Thu, May 24, 2012 at 10:11 PM, Andrew Trick <atrick at apple.com> wrote:

> I'd like to move the loop discovery algorithm out of the main header
> before I start changing it.
>
> This makes the LoopInfo interface readable, which I think is extremely
> important.
>
> Bonus:
> While I'm working on it, it speeds up my rebuilds by 4.7x.
> And for everyone else working outside LoopInfo, well your builds could be
> a full second faster ;-)
>
> We've been using the same explicit instantiation strategy in
> BlockFrequency for a year now and no one has complained. But I thought wait
> for feedback anyway--hopefully just doxygen nits!
>

If the strategy is to use explicit instantiation only, why not move it
completely into a .cc file?


>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20120524/0a50fe8c/attachment.html>


More information about the llvm-commits mailing list