This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: *ping* Re: [Patch, fortran] merge two gfc_trans_scalar_assign flags
- From: Paul Richard Thomas <paul dot richard dot thomas at gmail dot com>
- To: Mikael Morin <mikael dot morin at sfr dot fr>
- Cc: gfortran <fortran at gcc dot gnu dot org>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 6 Aug 2015 12:15:00 +0200
- Subject: Re: *ping* Re: [Patch, fortran] merge two gfc_trans_scalar_assign flags
- Authentication-results: sourceware.org; auth=none
- References: <5533C1BD dot 4000707 at sfr dot fr> <55C331CC dot 4050803 at sfr dot fr>
Dear Mikael,
Well spotted! This is fine for trunk, since it is 'obvious' once seen.
Cheers
Paul
On 6 August 2015 at 12:07, Mikael Morin <mikael.morin@sfr.fr> wrote:
> Le 19/04/2015 16:54, Mikael Morin a Ãcrit :
>>
>> Hello,
>>
>> while working on pr65792, I noticed that gfc_trans_scalar_assign's
>> l_is_temp and dealloc flags are used only once, and at the same place.
>> This patch merges them together.
>> The calls are changed from
>> gfc_trans_scalar_assign (...blah..., foo, x, bar);
>> to
>> gfc_trans_scalar_assign (...blah..., x, !foo && bar);
>> As foo is most of the time either true or false, this is a simplification.
>>
>> regression tested on x86-64-unknown-linux-gnu.
>> OK for post 5.1 trunk?
>>
>
> Ping: https://gcc.gnu.org/ml/gcc-patches/2015-04/msg00966.html
--
Outside of a dog, a book is a man's best friend. Inside of a dog it's
too dark to read.
Groucho Marx