llvm dev - Oct 2009 - [LLVMdev] llvm build errors on windows/mingw32

Well, Now I get the solution.
Please, someone can apply the patch.

Just modify the makefile resident in lib/System

But, a new error is appeared
llvm[2]: Linking Debug unit test ADT
d:/objs/llvm/msys/Debug/lib/libLLVMSystem.a(Process.o):e:/Downloads/llvm-trunk/lib/System/Win32/Process.inc:78:
undefined reference to `GetProcessMemoryInfo at 12'
collect2: ld returned 1 exit status
make[2]: *** [Debug/ADTTests.exe] Error 1
make[2]: Leaving directory `/d/objs/llvm/msys/unittests/ADT'
make[1]: *** [ADT/.makeall] Error 2
make[1]: Leaving directory `/d/objs/llvm/msys/unittests'
make: *** [all] Error 1


I don't know why, And Pijnacker, do you get the same error?
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20091002/04cc4be6/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: path-for-msys.patch
Type: application/octet-stream
Size: 317 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20091002/04cc4be6/attachment.obj>

"罗勇刚(Yonggang Luo)"  <luoyonggang at gmail.com> writes:
> Well, Now I get the solution.
> Please, someone can apply the patch.
>
> Just modify the makefile resident in lib/System
I'm afraid this patch is not acceptable. It activates C++ exception
support unconditionally while building libLLVMSystem, which is used
everywhere.

You'll better investigate why exception support is required for MSYS
while all other configure-based builds work fine without it.

[snip]

-- 
Óscar

>
>
> I'm afraid this patch is not acceptable. It activates C++ exception
> support unconditionally while building libLLVMSystem, which is used
> everywhere.
>
> You'll better investigate why exception support is required for MSYS
> while all other configure-based builds work fine without it.
>
> Sorry, I got no idea to solve this problem, do you have a method to get it
done?
Yours
   sincerely,
Yonggang Luo
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20091002/653f4787/attachment.html>

> Well, Now I get the solution.
> Please, someone can apply the patch.
> 
> Just modify the makefile resident in lib/System
> 
> But, a new error is appeared
> llvm[2]: Linking Debug unit test ADT
>
d:/objs/llvm/msys/Debug/lib/libLLVMSystem.a(Process.o):e:/Downloads/llvm-trunk/lib/System/Win32/Process.inc:78:
> undefined reference to `GetProcessMemoryInfo at 12'
> collect2: ld returned 1 exit status
> make[2]: *** [Debug/ADTTests.exe] Error 1
> make[2]: Leaving directory `/d/objs/llvm/msys/unittests/ADT'
> make[1]: *** [ADT/.makeall] Error 2
> make[1]: Leaving directory `/d/objs/llvm/msys/unittests'
> make: *** [all] Error 1
> 
> 
> I don't know why, And Pijnacker, do you get the same error?
Yes, I get the same error.

Ronald

> You'll better investigate why exception support is required for MSYS
> while all other configure-based builds work fine without it.
Surely it's required for any system that uses
lib/System/Win32/Signals.inc, because there's code in that file that
uses try/catch.

MingW is the only system I know of that uses that file; Cygwin
pretends to be Unix-like, so it uses lib/System/Unix/Signals.inc
instead. So I'd suggest the attached (untested) patch.

Jay.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libsystem.patch
Type: application/octet-stream
Size: 343 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20091002/b196319e/attachment.obj>

On 2009-10-02 01:07, 罗勇刚(Yonggang Luo)  wrote:
> Well, Now I get the solution.
> Please, someone can apply the patch.
> 
> Just modify the makefile resident in lib/System
> 
> But, a new error is appeared
> llvm[2]: Linking Debug unit test ADT
>
d:/objs/llvm/msys/Debug/lib/libLLVMSystem.a(Process.o):e:/Downloads/llvm-trunk/lib/System/Win32/Process.inc:78:
> undefined reference to `GetProcessMemoryInfo at 12'
GetProcessMemoryInfo should be defined in psapi.lib, but AFAICS mingw32
doesn't have this import lib.  It is included in the MS Platform SDK,
but I'm not sure whether you can link that into mingw32 programs...

> On 2009-10-02 01:07, 罗勇刚(Yonggang Luo)  wrote:
> > Well, Now I get the solution.
> > Please, someone can apply the patch.
> > 
> > Just modify the makefile resident in lib/System
> > 
> > But, a new error is appeared
> > llvm[2]: Linking Debug unit test ADT
> >
d:/objs/llvm/msys/Debug/lib/libLLVMSystem.a(Process.o):e:/Downloads/llvm-trunk/lib/System/Win32/Process.inc:78:
> > undefined reference to `GetProcessMemoryInfo at 12'
> 
> GetProcessMemoryInfo should be defined in psapi.lib, but AFAICS mingw32
> doesn't have this import lib.  It is included in the MS Platform SDK,
> but I'm not sure whether you can link that into mingw32 programs...
 
msdn suggests kernel32, but that is already included.

Ronald