<div class="gmail_quote">On Tue, Jul 12, 2011 at 1:47 AM, Francois Pichet <span dir="ltr"><<a href="mailto:pichet2000@gmail.com">pichet2000@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div><div></div><div class="h5">On Tue, Jul 12, 2011 at 2:52 AM, Chris Lattner <<a href="mailto:sabre@nondot.org" class="cremed">sabre@nondot.org</a>> wrote:<br>
> Author: lattner<br>
> Date: Tue Jul 12 01:52:18 2011<br>
> New Revision: 134972<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=134972&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=134972&view=rev</a><br>
> Log:<br>
> Fix a problem Eli ran into where we now reject incomplete arrays of<br>
> uncompleted struct types.  We now do what llvm-gcc does and compile<br>
> them into [i8 x 0].  If the type is later completed, we make sure that<br>
> it is appropriately cast.<br>
><br>
> We compile the terrible example to something like this now:<br>
><br>
> %struct.A = type { i32, i32, i32 }<br>
><br>
> @g = external global [0 x i8]<br>
><br>
> define void @_Z1fv() nounwind {<br>
> entry:<br>
>  call void @_Z3fooP1A(%struct.A* bitcast ([0 x i8]* @g to %struct.A*))<br>
>  ret void<br>
> }<br>
><br>
> declare void @_Z3fooP1A(%struct.A*)<br>
><br>
> define %struct.A* @_Z2f2v() nounwind {<br>
> entry:<br>
>  ret %struct.A* getelementptr inbounds ([0 x %struct.A]* bitcast ([0 x i8]* @g to [0 x %struct.A]*), i32 0, i64 1)<br>
> }<br>
><br>
><br>
><br>
><br>
> Modified:<br>
>    cfe/trunk/lib/CodeGen/CGExpr.cpp<br>
>    cfe/trunk/lib/CodeGen/CodeGenTypes.cpp<br>
>    cfe/trunk/test/CodeGenCXX/init-incomplete-type.cpp<br>
<br>
</div></div>This doesn't compile on Release mode with MSVC:<br>
4>CGExpr.cpp(1295): error C2664: 'EmitBitCastOfLValueToProperType' :<br>
cannot convert parameter 1 from 'clang::CodeGen::CGBuilderTy' to<br>
'llvm::IRBuilder<> &'<br>
4>CGExpr.cpp(1354): error C2664: 'EmitBitCastOfLValueToProperType' :<br>
cannot convert parameter 1 from 'clang::CodeGen::CGBuilderTy' to<br>
'llvm::IRBuilder<> &'<br>
4>CGExpr.cpp(1855): error C2664: 'EmitBitCastOfLValueToProperType' :<br>
cannot convert parameter 1 from 'clang::CodeGen::CGBuilderTy' to<br>
'llvm::IRBuilder<> &'<br>
<br>
Because on release CGF.Builder is actually a llvm::IRBuilder<false><br>
<div><div></div><div class="h5"></div></div></blockquote></div><br><div>I committed a fix in r134977. Not sure its really the best one, but it matches the surrounding static helper's argument forms, and it works in release builds.</div>