[PATCH] D52421: [Sema] Diagnose parameter names that shadow inherited field names

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 24 10:37:16 PDT 2018


aaron.ballman added inline comments.


================
Comment at: lib/Sema/SemaDecl.cpp:12380-12382
   }
 
+  if (LangOpts.CPlusPlus && II) {
----------------
lebedev.ri wrote:
> I think you could move it into the `if()` above?
You are correct, I'll hoist it.


https://reviews.llvm.org/D52421





More information about the cfe-commits mailing list