search for: ellvmdev

Hi, Daniel: Thanks a lot for detailed background information, we are willing to provide the right fix, however it will take time, do you mind if you forward me the discussion you had 5 months ago? I may not be able to access it since I only joined ellvmdev list this week. I did some performance measurement last night, some of our critical benchmark degraded up to 30% with your patch, so we have to turn it off for now at least. I posted patch to add a debug option (off by default), so we could turn it off with that option, could you please review it...

...rote: > > Hi, Daniel: > > > > Thanks a lot for detailed background information, we are willing to > provide the right fix, however it will take time, do you mind if you > forward me the discussion you had 5 months ago? I may not be able to > access it since I only joined ellvmdev list this week. > > > > I did some performance measurement last night, some of our critical > benchmark degraded up to 30% with your patch, so we have to turn it off for > now at least. > > > > I posted patch to add a debug option (off by default), so we could turn &gt...

I thought about a little bit more, I think adding Register pressure control in your patch or PRE may be the only choice. Because at least for this case I am looking at, what your patch did is created more relatively complex long live range, rematerialization is not smart enough to undo your change or at least without a lot of work, coalescing only create even longer live range not shorter,