[llvm-commits] PATCH: Enable scalar store promotion in DAG Legalizer
Tom Stellard
tom at stellard.net
Fri Dec 7 14:20:35 PST 2012
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>
More information about the llvm-commits
mailing list