On 2014-06-16, 12:14 PM, Ramana Radhakrishnan wrote:
On Mon, Jun 16, 2014 at 5:11 PM, Vladimir Makarov <vmaka...@redhat.com> wrote:

Any reason the check couldn't be like the earlier patch ?

i.e. else if (targetm.spill_class)
  {

....

}


I've just preferred to put most conditions in one place. That is the only reason. But if you want you can commit your variant.

I would have approved your patch, if I read your email before. I was in hurry to fix it as the original patch broke most LRA targets (ones without the hook definition) and the mistake itself was embarrassing for me.

Reply via email to