This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Combine location with block using block_locations

From: Michael Matz <matz at suse dot de>
To: Martin Jambor <mjambor at suse dot cz>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Guenther <richard dot guenther at gmail dot com>, Dehao Chen <dehao at google dot com>, Jan Hubicka <hubicka at ucw dot cz>, David Li <davidxl at google dot com>
Date: Wed, 19 Sep 2012 16:00:18 +0200 (CEST)
Subject: Re: [PATCH] Combine location with block using block_locations
References: <CAFiYyc1pA9+=_OE50rCSmKG4ftasrNYE8jRMt3gf_F+6ee9OVw@mail.gmail.com> <alpine.LNX.2.00.1209111500200.9940@wotan.suse.de> <CAFiYyc0vM+LZJ_6MRFud9c1LhyFsu2jf-E38FuDryoMo5G7iYw@mail.gmail.com> <CAO2gOZV8SP-sOVEJBEsvF+M3yiKwtw7yWarfhZ1xeCVYjE_7KA@mail.gmail.com> <alpine.LNX.2.00.1209111728160.9940@wotan.suse.de> <CAFiYyc2zQZwnGKtSxq5VGgQkN1TpSobwq9wt8GiwtwHKoWuOyA@mail.gmail.com> <alpine.LNX.2.00.1209121411120.9940@wotan.suse.de> <CAFiYyc0gqxhKPC_n+sT2U6GY-_uCFiRduvmsuJnp+JTjVip-tQ@mail.gmail.com> <alpine.LNX.2.00.1209121523200.9940@wotan.suse.de> <alpine.LNX.2.00.1209121612000.9940@wotan.suse.de> <20120919094040.GA3014@virgil.arch.suse.de>

Hi,
On Wed, 19 Sep 2012, Martin Jambor wrote:

> (The patch does not introduce any of the asserts Michael's patch had 
> because, as far as I my grep told me, IS_UNKNOWN_LOCATION is not in 
> trunk yet and I suppose the pre-approval does not cover introducing 
> things like that.)

Dehaos patch contains it.  For current trunk it would simply be an 
equality test with UNKNOWN_LOCATION.  If have no opinion if the asserts 
should be there or not.

Ciao,
Michael.

Follow-Ups:
- Re: [PATCH] Combine location with block using block_locations
  - From: Dehao Chen

References:
- Re: [PATCH] Combine location with block using block_locations
  - From: Richard Guenther
- Re: [PATCH] Combine location with block using block_locations
  - From: Michael Matz
- Re: [PATCH] Combine location with block using block_locations
  - From: Richard Guenther
- Re: [PATCH] Combine location with block using block_locations
  - From: Michael Matz
- Re: [PATCH] Combine location with block using block_locations
  - From: Richard Guenther
- Re: [PATCH] Combine location with block using block_locations
  - From: Michael Matz
- Re: [PATCH] Combine location with block using block_locations
  - From: Richard Guenther
- Re: [PATCH] Combine location with block using block_locations
  - From: Michael Matz
- Re: [PATCH] Combine location with block using block_locations
  - From: Michael Matz
- Re: [PATCH] Combine location with block using block_locations
  - From: Martin Jambor

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]