================
@@ -2495,10 +2471,100 @@ static FixItList fixVariableWithSpan(const VarDecl *VD,
   return fixLocalVarDeclWithSpan(VD, Ctx, getUserFillPlaceHolder(), Handler);
 }
 
+static FixItList fixVarDeclWithArray(const VarDecl *D, const ASTContext &Ctx,
+                                     UnsafeBufferUsageHandler &Handler) {
+  FixItList FixIts{};
+
+  if (auto CAT = dyn_cast<clang::ConstantArrayType>(D->getType())) {
----------------
ziqingluo-90 wrote:

nitpick:  can we replace this check with an assertion?   Because it is already 
checked at the caller:
```
 case FixitStrategy::Kind::Array: {
    if (VD->isLocalVarDecl() && isa<clang::ConstantArrayType>(VD->getType()))
      return fixVariableWithArray(VD, Tracker, Ctx, Handler);
```

https://github.com/llvm/llvm-project/pull/80084
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to