[PATCH] Allow the RegisterCoalescer to remove copies to reserved registers

hfinkel at anl.gov hfinkel at anl.gov
Wed Jan 14 15:39:41 PST 2015


In http://reviews.llvm.org/D6978#108879, @qcolombet wrote:

> > Perhaps we should restrict to the case of a reserved physical register?
>
>
> Reserved registers are a different story and in that case, by all means, this is a good idea :).


Good, will do :)

One more thing, I had asked:

> One thing that is not clear to me: Do I need to do anything special to update the 'dead def' live range corresponding to the updated PHYSREG?


Do you know if I need to do anything specific here, or will the register replacement code that already exists take care of this in a sufficient way?

Thanks again!

> 

> 

> > I can certainly try that if you don't like my suggestion above.

> 

> 

> I like your suggestion, the copy propagation improvement is just another angle of attack I was envisioning. It may be generally useful, but if we do not have motivating example, I would leave that for later.

> 

> Thanks,

> -Quentin



http://reviews.llvm.org/D6978

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list