cwalls251 / iphone-dev

Automatically exported from code.google.com/p/iphone-dev
0 stars 0 forks source link

llvm-convert.cpp broken #59

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
The "iPhone" version of llvm-convert.cpp is not compatible with the latest 
llvm-gcc trunk and produces compiling errors. It would be safer to use 
diff/patch instead.

Original issue reported on code.google.com by tim.schu...@gmail.com on 24 Sep 2007 at 12:05

GoogleCodeExporter commented 8 years ago
Just followed the instruction to build the toolchain on OS X 10.4.10 on G4. 
Have a compilation problem with 
llvm-convert.cpp. Seems that it is in fact a problem that can be simply 
resolved by modifying llvm-internal.h, 
line 562, by removing the third argument (const char *LongDoubleName).

Original comment by kyleg...@gmail.com on 24 Sep 2007 at 4:16

GoogleCodeExporter commented 8 years ago
Sigh, I knew this was going to happen. Diffs/patches aren't really the way to 
go, because Subversion doesn't 
support patches against external repositories; instead I will freeze the 
LLVM-GCC and advise people not to use 
the latest trunk. We have to wait on Google to get around to giving us extra 
space; this problem really stems 
from the absurdly low 100 MB quota they give us (2 GB for Gmail and 100 MB for 
googlecode - what were they 
thinking?!)

Original comment by nightwat...@gmail.com on 24 Sep 2007 at 5:50

GoogleCodeExporter commented 8 years ago
Wouldn't it be possible to do the following in a makefile?

1) detect if llvm-gcc-4.0 directory exists, if not then do an svn checkout of 
llvm-
gcc, else do an svn revert/update of llvm-gcc
2) apply llvm-gcc iphone diffs to working copy of llvm-gcc

Thanks.

Original comment by tim.schu...@gmail.com on 24 Sep 2007 at 7:08

GoogleCodeExporter commented 8 years ago
It's not really feasible, because diffs are not very human-modifiable. It would 
be easy for everyone else but very 
difficult for me and other developers to work on LLVM-GCC.

Original comment by nightwat...@gmail.com on 25 Sep 2007 at 1:35

GoogleCodeExporter commented 8 years ago
Should be fixed as soon as this commit gets through, which includes a 
slimmed-down version of LLVM-GCC but 
is taking an hour so far.

Original comment by nightwat...@gmail.com on 25 Sep 2007 at 4:19

GoogleCodeExporter commented 8 years ago
I just encountered this error after a fresh checkout of everything, so 
something still hasn't taken.  I edited llvm-
internal.h as suggested above and it seems to be working.

Original comment by tomsm...@gmail.com on 25 Sep 2007 at 6:24

GoogleCodeExporter commented 8 years ago
Should be fixed. I deleted my tree and checked everything out anew and it built 
just fine, so we should be good 
now.

Original comment by nightwat...@gmail.com on 26 Sep 2007 at 1:37