llvm dev - Apr 2014 - [LLVMdev] [patch] mergefunc struct result

When using the merge func pass with two functions that return the same  
kind of struct but with different element types merge func tries to do a  
bitcast of struct to struct.

%kv1 = type { i32*, i32* }
%kv2 = type { i8*, i8* }

define %kv1 @fn1() {
...
}

define %kv2 @fn2() {
..
}

The attached patch checks for two struct types & uses insert/extract to  
create the right record.


-- 
Carlo Kok
RemObjects Software
-------------- next part --------------
A non-text attachment was scrubbed...
Name: llvm-mergefunc-struct-result.diff
Type: application/octet-stream
Size: 1149 bytes
Desc: not available
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20140418/36179f05/attachment.obj>

a) this should go to llvm-commits to increase it's odds of being 
reviewed and applied
b) you'll need to provide a full test case - both because we strong 
encourage one for every patch and because it's not entirely clear what 
you're getting at (at least to me)

Philip

On 04/18/2014 05:19 AM, Carlo Kok wrote:
>
> When using the merge func pass with two functions that return the same 
> kind of struct but with different element types merge func tries to do 
> a bitcast of struct to struct.
>
> %kv1 = type { i32*, i32* }
> %kv2 = type { i8*, i8* }
>
> define %kv1 @fn1() {
> ...
> }
>
> define %kv2 @fn2() {
> ..
> }
>
> The attached patch checks for two struct types & uses insert/extract 
> to create the right record.
>
>
>
>
> _______________________________________________
> LLVM Developers mailing list
> LLVMdev at cs.uiuc.edu         http://llvm.cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20140418/e1c68adf/attachment.html>