[PATCH] D47297: [Modules][ObjC] Add protocol redefinition to the current scope/context

[Alex Lorenz via Phabricator via cfe-commits]

arphaman added inline comments.


================
Comment at: lib/Sema/SemaDeclObjC.cpp:1208
+    // serialize something meaningful.
+    if (getLangOpts().Modules)
+      PushOnScopeChains(PDecl, TUScope);
----------------
Is this a problem only for modules or does this show up in PCHs too? What would be the cost of using PushOnScopeChains unconditionally?


Repository:
  rC Clang

https://reviews.llvm.org/D47297