The comment here is not wrong, just it would be better if mentioning
the C++ front-end instead of just the nested function lowering.

gcc/ChangeLog:

        * cfgexpand.cc (add_scope_conflicts_1): Expand comment
        on when non-var clobbers show up.

Signed-off-by: Andrew Pinski <quic_apin...@quicinc.com>
---
 gcc/cfgexpand.cc | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gcc/cfgexpand.cc b/gcc/cfgexpand.cc
index f32cf1b20c9..6c1096363af 100644
--- a/gcc/cfgexpand.cc
+++ b/gcc/cfgexpand.cc
@@ -639,8 +639,9 @@ add_scope_conflicts_1 (basic_block bb, bitmap work, bool 
for_conflict)
        {
          tree lhs = gimple_assign_lhs (stmt);
          unsigned *v;
-         /* Nested function lowering might introduce LHSs
-            that are COMPONENT_REFs.  */
+         /* Handle only plain var clobbers.
+            Nested functions lowering and C++ front-end inserts clobbers
+            which are not just plain variables.  */
          if (!VAR_P (lhs))
            continue;
          if (DECL_RTL_IF_SET (lhs) == pc_rtx
-- 
2.34.1

Reply via email to