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,fortran]: fix for PR 33395

Tobias Burnus wrote:
> Note: In
>
>    if (!(expr || pointer))
>      return NULL_TREE;
> [...]
>    if (expr != NULL && expr->ts.type == BT_DERIVED
>
>
> I removed the "expr != NULL &&" as this can never happen due to the
> first if.
>   
I am really stupid!

The first one does not test "if(expr == NULL || pointer == NULL)" but
"if(expr == NULL && pointer == NULL)".

I now reverted the change - Rev. 128422.

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