[llvm-commits] [llvm] r153604 - in /llvm/trunk: lib/Analysis/IPA/GlobalsModRef.cpp test/Analysis/GlobalsModRef/pr12351.ll

[Chris Lattner]

On Mar 28, 2012, at 4:29 PM, Eli Friedman wrote:

> On Wed, Mar 28, 2012 at 3:58 PM, Chris Lattner <clattner at apple.com> wrote:
>> 
>> On Mar 28, 2012, at 2:31 PM, Rafael Espindola wrote:
>> 
>>> Author: rafael
>>> Date: Wed Mar 28 16:31:24 2012
>>> New Revision: 153604
>>> 
>>> URL: http://llvm.org/viewvc/llvm-project?rev=153604&view=rev
>>> Log:
>>> Handle intrinsics in GlobalsModRef. Fixes pr12351.
>> 
>> Looks great, it's scary that has been broken for this long.
> 
> Probably because LTO gets much less testing; we don't use
> GlobalsModRef for regular compilation.

Yes, which is sad.  We should really be using it at -O3.  It would also be really cool to enhance it to propagate up mod/ref TBAA type tags, which would also be a big win for C++.

-Chris