[llvm-commits] [llvm] r119040 - /llvm/trunk/unittests/CMakeLists.txt

[Francois Pichet]

On Sun, Nov 14, 2010 at 5:34 PM, NAKAMURA Takumi <geek4civic at gmail.com> wrote:
> Good morning, Francois.
>
> 2010/11/15 Francois Pichet <pichet2000 at gmail.com>:
>> ValueMapTest.cpp doesn't compile on VS2008 btw.. it generates all kind
>> of weird errors about this.
>
> Indeed, also on vs8, I met many errors.
> Fortunately, this issue did not affect to buildbots.

I investigated and the problem is clearly a documented MSVC 2008 bug.
see:
http://connect.microsoft.com/VisualStudio/feedback/details/331418/erroneous-error-c2355-this-can-only-be-referenced-inside-non-static-member-functions

The workaround is to remove the keyword "this" from the lines MSVC can't handle.
If there is no objection.. i'll make that change.