]> git.lizzy.rs Git - rust.git/commit
Rollup merge of #55802 - wesleywiser:inlined_calls_2_electric_boogaloo, r=nagisa
authorPietro Albini <pietro@pietroalbini.org>
Sat, 10 Nov 2018 23:21:19 +0000 (00:21 +0100)
committerGitHub <noreply@github.com>
Sat, 10 Nov 2018 23:21:19 +0000 (00:21 +0100)
commit5b0b0ce61ebf0da19d89f1be7ccea67f6cd128a7
tree3f040a23fb74bb89768b399b888e0d49749b84ab
parent18195d413333d8d9ab5d1ed6a5af2993c0c8e5d8
parent3cce5c79778088a26f6099f293256d5d7834fdb3
Rollup merge of #55802 - wesleywiser:inlined_calls_2_electric_boogaloo, r=nagisa

Don't inline virtual calls (take 2)

When I fixed the previous mis-optimizations, I didn't realize there were
actually two different places where we mutate `callsites` and both of
them should have the same behavior.

As a result, if a function was inlined and that function contained
virtual function calls, they were incorrectly being inlined. I also
added a test case which covers this.