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.mo...@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