[PATCH] D52384: [Sema] Fix redeclaration contexts for enumerators in C

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Sep 28 09:18:16 PDT 2018


aaron.ballman added inline comments.


================
Comment at: lib/AST/DeclBase.cpp:1711-1712
+  // contexts are always skipped.
+  while (SkipRecords ? Ctx->isTransparentContext() || Ctx->isRecord()
+                     : Ctx->isTransparentContext())
     Ctx = Ctx->getParent();
----------------
majnemer wrote:
> Seems simpler as:
>   while ((SkipRecords && Ctx->isRecord()) || Ctx->isTransparentContext())
Agreed; I've changed it.


https://reviews.llvm.org/D52384





More information about the cfe-commits mailing list