[PATCH] D46165: [Modules] Handle ObjC/C ODR-like semantics for EnumConstantDecl

Bruno Cardoso Lopes via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 30 14:24:33 PDT 2018


bruno marked an inline comment as done.
bruno added inline comments.


================
Comment at: lib/Serialization/ASTReaderDecl.cpp:2568-2570
   // ODR-based merging is only performed in C++. In C, identically-named things
   // in different translation units are not redeclarations (but may still have
+  // compatible types). However, clang is able to merge tag types with ODR-like
----------------
rsmith wrote:
> I think this comment would benefit from more rewriting, since we've basically decided that we do want to apply the ODR merging rule to C.
Sure, will update that!


Repository:
  rC Clang

https://reviews.llvm.org/D46165





More information about the cfe-commits mailing list