Re: [PATCH 01/10] Fix IRA ICE.

2018-12-10 Thread Andrew Stubbs
On 08/12/2018 16:23, Jeff Law wrote: On 12/8/18 5:14 AM, Richard Sandiford wrote: Andrew Stubbs writes: On 21/11/2018 00:47, Jeff Law wrote: This seems like a really gross hack and sets an expectation that generating registers in the target after IRA has started is OK. It is not OK. THe

Re: [PATCH 01/10] Fix IRA ICE.

2018-12-08 Thread Jeff Law
On 12/8/18 5:14 AM, Richard Sandiford wrote: > Andrew Stubbs writes: >> On 21/11/2018 00:47, Jeff Law wrote: >>> This seems like a really gross hack and sets an expectation that >>> generating registers in the target after IRA has started is OK. It is >>> not OK. THe fact that this works is,

Re: [PATCH 01/10] Fix IRA ICE.

2018-12-08 Thread Richard Sandiford
Andrew Stubbs writes: > On 21/11/2018 00:47, Jeff Law wrote: >> This seems like a really gross hack and sets an expectation that >> generating registers in the target after IRA has started is OK. It is >> not OK. THe fact that this works is, IMHO, likely an accident. > > What's the proper test

Re: [PATCH 01/10] Fix IRA ICE.

2018-11-21 Thread Andrew Stubbs
On 21/11/2018 00:47, Jeff Law wrote: This seems like a really gross hack and sets an expectation that generating registers in the target after IRA has started is OK. It is not OK. THe fact that this works is, IMHO, likely an accident. What's the proper test for this? Neither lra_in_progress

Re: [PATCH 01/10] Fix IRA ICE.

2018-11-20 Thread Jeff Law
On 11/16/18 9:27 AM, Andrew Stubbs wrote: > > This patch is unchanged from that which was posted before. Discussion > fizzled out there and I was too busy with other patches to restart it > then. This issue needs to be resolved before libgfortran can be > compiled for GCN. > > The IRA pass

[PATCH 01/10] Fix IRA ICE.

2018-11-16 Thread Andrew Stubbs
This patch is unchanged from that which was posted before. Discussion fizzled out there and I was too busy with other patches to restart it then. This issue needs to be resolved before libgfortran can be compiled for GCN. The IRA pass makes an assumption that any pseudos created after the pass