[PATCH] D102241: [clang] p1099 4/5: using enum EnumTag

Gabor Marton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 18 02:08:56 PDT 2021


martong added inline comments.


================
Comment at: clang/lib/AST/ASTImporter.cpp:4654
+
+  return ImportUsingShadowDecls(D, ToUsingEnum);
 }
----------------
Could it work if you imported the shadow declarations **before** creating the `UsingEnumDecl`? I yes, then that would be the preferred approach b/c that would fix the `// FIXME: We return error here but the definition is already created`.


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

https://reviews.llvm.org/D102241



More information about the cfe-commits mailing list