[cfe-dev] [llvm-commits] [llvm] r158787 - in /llvm/trunk: include/llvm/Analysis/LoopInfo.h include/llvm/Analysis/LoopInfoImpl.h lib/Analysis/LoopInfo.cpp lib/CodeGen/MachineLoopInfo.cpp

Kim Gräsman kim.grasman at gmail.com
Wed Jun 20 04:37:14 PDT 2012


On Wed, Jun 20, 2012 at 8:12 AM, Andrew Trick <atrick at apple.com> wrote:
>
> At least for compilers that support it putting the various things like:
>
> __extension__ extern template class LoopInfoBase<BasicBlock, Loop>;
>
> Into the header would make me happier...
>
> I'm OK relying on compilers to make this "just work" if they don't support
> the extension (MSVC likely, dunno), but where the do, this will make sure
> everything works together nicely.

I'm almost sure MSVC is fine with extern template; I didn't know the
language feature existed until about a year ago, and my experiments
seemed to indicate it worked fine in VC 10.

FWIW,
- Kim



More information about the cfe-dev mailing list