[patch] fix loopvectorizer bug with multiple address spaces PR16306

[Nadav Rotem]

Pekka, 

LGTM.  Please also make sure that we are not dropping the address space for Loads. 

Thanks,
Nadav

On Jun 17, 2013, at 10:57 AM, Eli Friedman <eli.friedman at gmail.com> wrote:

> Forwarding to the correct list.
> 
> ---------- Forwarded message ----------
> From: Pekka Jääskeläinen <pekka.jaaskelainen at tut.fi>
> Date: Mon, Jun 17, 2013 at 8:06 AM
> Subject: [patch] fix loopvectorizer bug with multiple address spaces PR16306
> To: cfe-commits at cs.uiuc.edu
> 
> 
> Hello,
> 
> Attached patch fixes a severe regression caused by LoopVectorizer when
> vectorizing loops with memory accesses to non-zero address spaces. It
> simply drops the AS info. It's a rather nasty problem as there is no
> way (at least known to me) to disable the loop vectorizer *only* from
> clang/opt command line.
> 
> Seems like an obvious fix.
> 
> http://llvm.org/bugs/show_bug.cgi?id=16306
> 
> The patch applies to trunk and 3.3 branches. Tom, could you apply
> this to the (upcoming?) 3.3 maintenance branch as well?
> 
> Thanks,
> -- 
> --Pekka
> 
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
> 
> 
> <loopvec-address-space-fix.patch>

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