On 3/12/2012 8:10 PM, Nick Sabalausky wrote: > "Brad Roberts" <bra...@puremagic.com> wrote: >> >> See also: bug 4440 >> >> The patch in there, if it hasn't bit rotten to badly (I suspect it has) >> will handle _this_ case. But almost no other >> case of inlining delegates. >> >> It'd be a good area for someone who wants an interesting and non-trivial >> problem to dig into. It wouldn't touch all >> that much of the codebase as the inliner is fairly self-contained. At >> least, that's what I recall from when I looked at >> this stuff a couple years ago. >> > > Do you think that patch would be a good starting place for further work, or > would a proper solution likely necessitate an entirely different approach? >
I suspect that, to fix it for a broader set of cases, it'll need much larger changes. During those changes the simple version in that patch would likely disappear. But take that all with a huge grain of salt.