[v2,0/2] LD: SEGMENT_START expression handling fixes

Message ID alpine.LFD.2.21.1809132337450.16413@eddie.linux-mips.org
Headers show
Series
  • LD: SEGMENT_START expression handling fixes
Related show

Message

Maciej W. Rozycki Sept. 13, 2018, 11:41 p.m.
Hi Alan,

 I decided to complete this change after all.  I split it into a small 
patch series as an update you requested is logically separate.  See 
individual patch descriptions for further details.

 As for historical reasons my personal development environment has quite 
an old version of GCC testing these changes revealed a couple of build 
issues across 3 targets.  These turned out straightforward to fix, so I'll 
be proposing them separately, as I believe we have no need to artificially 
limit support for older compiler versions if that does not cause extra
maintenance burden, which is the case here.

 With those fixes in place I saw no regressions with this patch series in 
testing across my usual targets.

 OK to apply?

  Maciej

Comments

Alan Modra Sept. 14, 2018, 12:02 a.m. | #1
On Fri, Sep 14, 2018 at 12:41:08AM +0100, Maciej W. Rozycki wrote:
> Hi Alan,

> 

>  I decided to complete this change after all.  I split it into a small 

> patch series as an update you requested is logically separate.  See 

> individual patch descriptions for further details.

> 

>  As for historical reasons my personal development environment has quite 

> an old version of GCC testing these changes revealed a couple of build 

> issues across 3 targets.  These turned out straightforward to fix, so I'll 

> be proposing them separately, as I believe we have no need to artificially 

> limit support for older compiler versions if that does not cause extra

> maintenance burden, which is the case here.

> 

>  With those fixes in place I saw no regressions with this patch series in 

> testing across my usual targets.

> 

>  OK to apply?


OK, thanks!

-- 
Alan Modra
Australia Development Lab, IBM
Maciej W. Rozycki Sept. 14, 2018, 7:26 p.m. | #2
On Fri, 14 Sep 2018, Alan Modra wrote:

> OK, thanks!


 Committed now, thanks for your review.

  Maciej