[clang] 552e465 - [Clang][NFCI] Fix a logic issue in TransformDesignatedInitExpr (#127211)

via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 16 05:35:00 PST 2025


Author: GS-GOAT
Date: 2025-02-16T21:34:56+08:00
New Revision: 552e4659d3f935b8896a1350856781417bcbdcde

URL: https://github.com/llvm/llvm-project/commit/552e4659d3f935b8896a1350856781417bcbdcde
DIFF: https://github.com/llvm/llvm-project/commit/552e4659d3f935b8896a1350856781417bcbdcde.diff

LOG: [Clang][NFCI] Fix a logic issue in TransformDesignatedInitExpr (#127211)

It was clearly a typo regarding whether the array index expression has changed in its transforming.

This doesn't appear to be a functional change in practice, so no test case or release note provided.

Fixes #126113

Added: 
    

Modified: 
    clang/lib/Sema/TreeTransform.h

Removed: 
    


################################################################################
diff  --git a/clang/lib/Sema/TreeTransform.h b/clang/lib/Sema/TreeTransform.h
index fc1e3f7d58f4d..73e979927b4f3 100644
--- a/clang/lib/Sema/TreeTransform.h
+++ b/clang/lib/Sema/TreeTransform.h
@@ -13672,7 +13672,7 @@ TreeTransform<Derived>::TransformDesignatedInitExpr(DesignatedInitExpr *E) {
       Desig.AddDesignator(
           Designator::CreateArrayDesignator(Index.get(), D.getLBracketLoc()));
 
-      ExprChanged = ExprChanged || Init.get() != E->getArrayIndex(D);
+      ExprChanged = ExprChanged || Index.get() != E->getArrayIndex(D);
       ArrayExprs.push_back(Index.get());
       continue;
     }


        


More information about the cfe-commits mailing list