search for: d11234

...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 and commit it for me if possible? I don't have commit right yet, will ask soon. http://reviews.llvm.org/D11234 Thanks again. Lawrence Hu -----Original Message----- From: Daniel Berlin [mailto:dberlin at dberlin.org] Sent: Wednesday, July 15, 2015 7:48 AM To: Lawrence Cc: LLVM Developers Mailing List Subject: Re: Register pressure mechanism in PRE or Smarter rematerialization/split/spiller/coalescing ?...

...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 and commit it for me if > possible? I don't have commit right yet, will ask soon. > > http://reviews.llvm.org/D11234 > > > > Thanks again. > > > > Lawrence Hu > > > > > > -----Original Message----- > > From: Daniel Berlin [mailto:dberlin at dberlin.org] > > Sent: Wednesday, July 15, 2015 7:48 AM > > To: Lawrence > > Cc: LLVM Developers Mailing List...

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,