[Webkit-unassigned] [Bug 133753] Change the order of the alias analysis passes to align with the opt pipeline of LLVM

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=133753


Andrew Trick <atrick at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |atrick at apple.com




--- Comment #4 from Andrew Trick <atrick at apple.com>  2014-06-12 20:27:00 PST ---
The AA implementations are queried in reverse order, last added, first queried.

For C++, we query BasicAA first for correctness. It may actually disagree with TBAA! If TBAA says objects can't alias, but they are obviously derived from the same pointer, then TBAA is wrong and we want to catch that.

For JS, that's not an issue and it makes sense to query TBAA first instead since it is probably faster.

However, if we can't measure any compile time difference, then I agree with Juergen, that it's a bit less risky to be consistent with the C++ optimizer.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.