[llvm-commits] PATCH: Enable scalar store promotion in DAG Legalizer

[Tom Stellard]

On Fri, Dec 07, 2012 at 06:49:35PM +0100, Duncan Sands wrote:
> Hi Tom,
> 
> >The attached patch removes an assert that prevents the DAG Legalizer
> >from promoting scalar stores.  I'm not sure why this assert is here,
> >but it seems safe to remove.
> 
> can you please add an assertion that the old and new types have the same
> number of bits.  Otherwise it seems OK to me.
> 

Hi Ducan,

Thanks for the review, here are the updated patches.

-Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: LegalizeDAG-Allow-promotion-of-scalar-loads.patch
Type: text/x-diff
Size: 849 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121207/db9d35ae/attachment.patch>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: LegalizeDAG-Allow-promotion-of-scalar-stores.patch
Type: text/x-diff
Size: 946 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121207/db9d35ae/attachment-0001.patch>