Re: [Patch LRA] PR 61522 - Handle NULL targetm.spill_class

2014-06-16 Thread Vladimir Makarov
On 2014-06-16, 12:14 PM, Ramana Radhakrishnan wrote: On Mon, Jun 16, 2014 at 5:11 PM, Vladimir Makarov 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 on

Re: [Patch LRA] PR 61522 - Handle NULL targetm.spill_class

2014-06-16 Thread Ramana Radhakrishnan
On Mon, Jun 16, 2014 at 5:11 PM, Vladimir Makarov wrote: > On 2014-06-16, 11:05 AM, Ramana Radhakrishnan wrote: >> >> Hi, >> >> This handles NULL targetm.spill_class in assign_by_spills. This >> showed up as a segfault during a build for arm-none-linux-gnueabi(hf). >> >> Fix pre-approved by r

Re: [Patch LRA] PR 61522 - Handle NULL targetm.spill_class

2014-06-16 Thread Vladimir Makarov
On 2014-06-16, 11:05 AM, Ramana Radhakrishnan wrote: Hi, This handles NULL targetm.spill_class in assign_by_spills. This showed up as a segfault during a build for arm-none-linux-gnueabi(hf). Fix pre-approved by richi on IRC , verified that bootstrap continues from where things broke furth

Re: [Patch LRA] PR 61522 - Handle NULL targetm.spill_class

2014-06-16 Thread Jakub Jelinek
On Mon, Jun 16, 2014 at 04:05:52PM +0100, Ramana Radhakrishnan wrote: > This handles NULL targetm.spill_class in assign_by_spills. This showed > up > as a segfault during a build for arm-none-linux-gnueabi(hf). > > Fix pre-approved by richi on IRC , verified that bootstrap continues from >