llvm dev - Jan 2009 - [LLVMdev] [PATCH] Build fails on windows with VC2009

Hi everybody,

I just updated to the latest svn trunk version, but now the code does not
compile anymore.
It trows an error in "f:\dev\llvm\lib\system\Win32/DynamicLibrary.inc(159)
:
error C2228: left of '.find' must have class/struct/union".


Below you can find the small patch which makes the compiler happy again :-)
Sincerely yours
Marius Wachtler

----------------------------------------

Index: lib/System/Win32/DynamicLibrary.inc
==================================================================---
lib/System/Win32/DynamicLibrary.inc    (revision 63322)
+++ lib/System/Win32/DynamicLibrary.inc    (working copy)
@@ -156,8 +156,8 @@

 void* DynamicLibrary::SearchForAddressOfSymbol(const char* symbolName) {
   // First check symbols added via AddSymbol().
-  std::map<std::string, void *>::iterator I = g_symbols.find(symbolName);
-  if (I != g_symbols.end())
+  std::map<std::string, void *>::iterator I =
g_symbols().find(symbolName);
+  if (I != g_symbols().end())
     return I->second;

   // Now search the libraries.
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20090129/93f647be/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fixBuild.patch
Type: application/octet-stream
Size: 642 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20090129/93f647be/attachment.obj>

On Jan 29, 2009, at 10:35 AM, Marius Wachtler wrote:
> Hi everybody,
>
> I just updated to the latest svn trunk version, but now the code  
> does not compile anymore.
> It trows an error in "f:\dev\llvm\lib\system\Win32/ 
> DynamicLibrary.inc(159) : error C2228: left of '.find' must have  
> class/struct/union".
Oops, sorry about that, fixed, please verify!
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090126/072886.html

-Chris

On Thu, Jan 29, 2009 at 7:53 PM, Chris Lattner <clattner at apple.com>
wrote:
>
> On Jan 29, 2009, at 10:35 AM, Marius Wachtler wrote:
>
> > Hi everybody,
> >
> > I just updated to the latest svn trunk version, but now the code
> > does not compile anymore.
> > It trows an error in "f:\dev\llvm\lib\system\Win32/
> > DynamicLibrary.inc(159) : error C2228: left of '.find' must
have
> > class/struct/union".
>
> Oops, sorry about that, fixed, please verify!
>
>
http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20090126/072886.html
>
> -Chris
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev

It works.

Thanks for applying my patch so fast.

- Marius Wachtler
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20090129/00e99536/attachment.html>