[PATCH] Fix for CodeGenPrepare wrongly updating the map of sunk addresses

Andrea_DiBiagio at sn.scee.net Andrea_DiBiagio at sn.scee.net
Tue May 7 02:38:16 PDT 2013


Hi Nick,
Thanks for the review!

Nick Lewycky <nicholas at mxc.ca> wrote on 05/05/2013 23:53:55:

> From: Nick Lewycky <nicholas at mxc.ca>
> 
> Great!
> 
> -    DenseMap<Value*, Value*> SunkAddrs;
> +    ValueMap<Value*,Value*> SunkAddrs;
> 
> Should be a space between the template arguments.
> 
> Thanks for the patch, please commit it!
> 
> Nick

I forgot to mention on my last email that I haven't got commit access to 
the repository.
Could you (or anybody else) submit this patch for me?

P.s.: you can find the updated patch to commit in attachment 
(I fixed the space between the template arguments as requested by Nick).

Thanks in advance,
Andrea Di Biagio



**********************************************************************
This email and any files transmitted with it are confidential and intended 
solely for the use of the individual or entity to whom they are addressed. 
If you have received this email in error please notify postmaster at scee.net
This footnote also confirms that this email message has been checked for 
all known viruses.
Sony Computer Entertainment Europe Limited
Registered Office: 10 Great Marlborough Street, London W1F 7LP, United 
Kingdom
Registered in England: 3277793
**********************************************************************

P Please consider the environment before printing this e-mail
-------------- next part --------------
A non-text attachment was scrubbed...
Name: patch-codegen-prepare.diff
Type: application/octet-stream
Size: 3214 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130507/9e8abbd3/attachment.obj>


More information about the llvm-commits mailing list