Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Reland "[vm/compiler] Fine tune operator and phi handling."
This reverts commit cde4793. Reason for revert: the real underlying bug has been fixed Original change's description: > Revert "[vm/compiler] Fine tune operator and phi handling." > > This reverts commit 4d5b5bb. > > Reason for revert: We are seeing an AOT compiler issue as a result of this change (please see flutter/flutter#23879) > > Original change's description: > > [vm/compiler] Fine tune operator and phi handling. > > > > Rationale: > > Running an extra call specialization pass a bit later > > in the compiler passes stream (after types are propagated) > > recognizes more operators, which execute more efficiently. > > In addition, unboxing phis seems useful on all archs, not > > just 32-bit. These minor tweeks combined improve the prime > > number benchmark 3 fold. > > > > flutter/flutter#19677 > > > > Change-Id: Ib2102ce807c2f0a9f801542e0c4bc6a7673f552b > > Reviewed-on: https://dart-review.googlesource.com/69240 > > Reviewed-by: Vyacheslav Egorov <[email protected]> > > Commit-Queue: Aart Bik <[email protected]> > > [email protected],[email protected],[email protected],[email protected] > > # Not skipping CQ checks because original CL landed > 1 day ago. > > Change-Id: I46e420f9355bfc3c4a3f2fb9274f648e8a596d9a > Reviewed-on: https://dart-review.googlesource.com/c/83229 > Reviewed-by: Siva Annamalai <[email protected]> > Reviewed-by: Aart Bik <[email protected]> > Commit-Queue: Siva Annamalai <[email protected]> [email protected],[email protected],[email protected],[email protected] # Not skipping CQ checks because original CL landed > 1 day ago. Change-Id: I7f9d892a6225c42810e7be52b6845bfee9cc017d Reviewed-on: https://dart-review.googlesource.com/c/83800 Reviewed-by: Aart Bik <[email protected]> Reviewed-by: Alexander Markov <[email protected]> Commit-Queue: Aart Bik <[email protected]>
- Loading branch information