[PATCH] D12064: [GMR] isNonEscapingGlobalNoAlias() should look through Bitcasts/GEPs when looking at loads.

hfinkel@anl.gov via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 17 01:46:49 PDT 2015


hfinkel added a comment.

In http://reviews.llvm.org/D12064#225516, @mkuper wrote:

> Thanks for the idea, Hal. 
>  But, unless I'm missing something, this doesn't quite work, because the base condition is different.
>
> Let's say you have:
>  %v = load i32, i32* @g1
>
> isNonEscapingGlobalNoAlias(@g1, @g1) should return false, but isNonEscapingGlobalNoAlias(@g1, %v) should return true. 
>  So we can't just push @g1 onto the worklist.


Good point.


http://reviews.llvm.org/D12064





More information about the llvm-commits mailing list