[PATCH] D30726: Do not cache AliasSetTracker even in the old PM

[Chandler Carruth via Phabricator via llvm-commits]

chandlerc added a comment.

In https://reviews.llvm.org/D30726#695172, @sanjoy wrote:

> In https://reviews.llvm.org/D30726#695160, @hfinkel wrote:
>
> > We're only caching to save on compile time, right? Is there any noticeable compile-time impact?
>
>
> I'll measure.


I think this is the critical thing.

My expectation had been to have to add caching back in the new PM to address compile time issues. I think we need to understand how much this impacts compile time.

Certainly, not caching things seems much easier to get correct. ;]


https://reviews.llvm.org/D30726