[PATCH] D128449: [clang] Introduce -Warray-parameter

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 8 08:37:12 PDT 2022


aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

LGTM, thank you for the new diagnostic!



================
Comment at: clang/lib/Sema/SemaDecl.cpp:3213
+static bool EquivalentArrayTypes(QualType Old, QualType New,
+                                 ASTContext const &Ctx) {
+
----------------
East const? MONSTEROUS! ;-) (We tend to use west const mostly in the code base.)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128449/new/

https://reviews.llvm.org/D128449



More information about the cfe-commits mailing list