Re: [PATCH] Fix promotion of const local arrays to static storage

2014-08-18 Thread Patrick Palka
On Mon, Aug 18, 2014 at 8:59 AM, Patrick Palka wrote: > On Mon, Aug 18, 2014 at 8:50 AM, Richard Biener > wrote: >> On Mon, Aug 18, 2014 at 2:31 PM, Patrick Palka wrote: >>> On Mon, Aug 18, 2014 at 6:48 AM, Richard Biener >>> wrote: On Mon, Aug 18, 2014 at 4:00 AM, Patrick Palka wro

Re: [PATCH] Fix promotion of const local arrays to static storage

2014-08-18 Thread Patrick Palka
On Mon, Aug 18, 2014 at 8:50 AM, Richard Biener wrote: > On Mon, Aug 18, 2014 at 2:31 PM, Patrick Palka wrote: >> On Mon, Aug 18, 2014 at 6:48 AM, Richard Biener >> wrote: >>> On Mon, Aug 18, 2014 at 4:00 AM, Patrick Palka wrote: Hi, The fix for PR38615 indirectly broke the promo

Re: [PATCH] Fix promotion of const local arrays to static storage

2014-08-18 Thread Richard Biener
On Mon, Aug 18, 2014 at 2:31 PM, Patrick Palka wrote: > On Mon, Aug 18, 2014 at 6:48 AM, Richard Biener > wrote: >> On Mon, Aug 18, 2014 at 4:00 AM, Patrick Palka wrote: >>> Hi, >>> >>> The fix for PR38615 indirectly broke the promotion of const local arrays >>> to static storage in many cases.

Re: [PATCH] Fix promotion of const local arrays to static storage

2014-08-18 Thread Patrick Palka
On Mon, Aug 18, 2014 at 6:48 AM, Richard Biener wrote: > On Mon, Aug 18, 2014 at 4:00 AM, Patrick Palka wrote: >> Hi, >> >> The fix for PR38615 indirectly broke the promotion of const local arrays >> to static storage in many cases. The commit in question, r143570, made >> it so that only arrays

Re: [PATCH] Fix promotion of const local arrays to static storage

2014-08-18 Thread Richard Biener
On Mon, Aug 18, 2014 at 4:00 AM, Patrick Palka wrote: > Hi, > > The fix for PR38615 indirectly broke the promotion of const local arrays > to static storage in many cases. The commit in question, r143570, made > it so that only arrays that don't potentially escape from the scope in > which they'r

[PATCH] Fix promotion of const local arrays to static storage

2014-08-17 Thread Patrick Palka
Hi, The fix for PR38615 indirectly broke the promotion of const local arrays to static storage in many cases. The commit in question, r143570, made it so that only arrays that don't potentially escape from the scope in which they're defined (i.e. arrays for which TREE_ADDRESSABLE is 0) are candid