[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

[Nico Weber]

On Wed, Jun 20, 2012 at 4:37 AM, Kim Gräsman <kim.grasman at gmail.com> wrote:
> 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.

It doesn't work in MSVC2008.

>
> FWIW,
> - Kim
> _______________________________________________
> cfe-dev mailing list
> cfe-dev at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev