[PATCH] fix for PR16393: miscompile with struct byval

Manman Ren mren at apple.com
Mon Jul 15 17:04:45 PDT 2013


In r 186350 (verifier) and r186364 (PEI).

Thanks Stepan for the extra testing case (forgot to mention it in the commit message).
Thanks for suggestions and reviewing.

Manman

On Jul 15, 2013, at 4:37 PM, Jim Grosbach wrote:

> Me too. Thanks!
> 
> On Jul 15, 2013, at 4:36 PM, Jakob Stoklund Olesen <jolesen at apple.com> wrote:
> 
>> 
>> On Jul 12, 2013, at 4:29 PM, Manman Ren <mren at apple.com> wrote:
>> 
>>> Updated patch to use SmallVector instead of DenseMap:
>>> --- lib/CodeGen/PrologEpilogInserter.cpp	(revision 186215)
>>> +++ lib/CodeGen/PrologEpilogInserter.cpp	(working copy)
>>> 
>> 
>> LGTM
>> 
>> Thanks,
>> /jakob
> 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130715/291a66ac/attachment.html>


More information about the llvm-commits mailing list