This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: SSA identifiers
- From: Richard Biener <richard dot guenther at gmail dot com>
- To: Andrew MacLeod <amacleod at redhat dot com>
- Cc: Jakub Jelinek <jakub at redhat dot com>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 27 Aug 2013 09:54:52 +0200
- Subject: Re: SSA identifiers
- Authentication-results: sourceware.org; auth=none
- References: <51CC8391 dot 5000900 at redhat dot com> <20130627183940 dot GG2336 at tucnak dot redhat dot com> <51CC89D8 dot 4080607 at redhat dot com>
On Thu, Jun 27, 2013 at 8:52 PM, Andrew MacLeod <amacleod@redhat.com> wrote:
> On 06/27/2013 02:39 PM, Jakub Jelinek wrote:
>>
>>
>> in tree-ssanames.c:release_ssa_names() :
>>
>> if (! SSA_NAME_IN_FREE_LIST (var))
>> {
>> tree saved_ssa_name_var = SSA_NAME_VAR (var);
>> int saved_ssa_name_version = SSA_NAME_VERSION (var);
>> use_operand_p imm = &(SSA_NAME_IMM_USE_NODE (var));
>> <..>
>> /* Hopefully this can go away once we have the new incremental
>> SSA updating code installed. */
>> SET_SSA_NAME_VAR_OR_IDENTIFIER (var, saved_ssa_name_var);
>> I don't see a big issue with this, sure, you could
>> tree saved_ssa_name_identifier = saved_ssa_name_var ? saved_ssa_name_var :
>> SSA_NAME_IDENTIFIER (var);
>> and use that instead in SET_SSA_NAME_VAR_OR_IDENTIFIER.
>
>
> Yeah I wasn't too concerned about this one, the outof-ssa case looked like
> more of a possible issue. Maybe neither is, they just popped out as
> inconsistent uses.
Restoring SSA_NAME_VAR_OR_IDENTIFIER is only for debugging. Yes,
it probably should save SSA_NAME_VAR_OR_IDENTIFIER instead of SSA_NAME_VAR.
Richard.
> Andrew
>