From: Marek Polacek Date: Mon, 20 Apr 2020 17:37:35 +0000 (-0400) Subject: coroutines: Fix STRIP_NOPS usage. X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=c5e4be6b3645fb2294296310a3705f8be0e75da8;p=gcc.git coroutines: Fix STRIP_NOPS usage. parm = STRIP_NOPS (parm); is unnecessary and generates warning: operation on 'parm' may be undefined [-Wsequence-point] when cp/coroutines.cc is compiled with -std=c++11. * coroutines.cc (captures_temporary): Don't assign the result of STRIP_NOPS to the same variable. --- diff --git a/gcc/cp/ChangeLog b/gcc/cp/ChangeLog index e210eb5b80e..891ec79fcfd 100644 --- a/gcc/cp/ChangeLog +++ b/gcc/cp/ChangeLog @@ -1,3 +1,8 @@ +2020-04-20 Marek Polacek + + * coroutines.cc (captures_temporary): Don't assign the result of + STRIP_NOPS to the same variable. + 2020-04-20 Nathan Sidwell PR 94454 - tpl-tpl-parms are not canonicalizable types diff --git a/gcc/cp/coroutines.cc b/gcc/cp/coroutines.cc index 0a8e7521c4f..ceb8daa4e46 100644 --- a/gcc/cp/coroutines.cc +++ b/gcc/cp/coroutines.cc @@ -2652,7 +2652,7 @@ captures_temporary (tree *stmt, int *do_subtree, void *d) parm = TREE_OPERAND (parm, 0); if (TREE_CODE (parm) == INDIRECT_REF) parm = TREE_OPERAND (parm, 0); - parm = STRIP_NOPS (parm); + STRIP_NOPS (parm); } /* This isn't a temporary. */