[PATCH] D122160: [clang][extract-api] Refactor ExtractAPI and improve docs

Zixu Wang via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Mar 24 15:13:43 PDT 2022


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


================
Comment at: clang/lib/ExtractAPI/Serialization/SymbolGraphSerializer.cpp:159
+  switch (Language) {
+  case Language::C:
+    return "c";
----------------
zixuw wrote:
> zixuan-wu wrote:
> > It's same name as `Language` variable above, and it cause compile error. @zixuw 
> > Maybe the error depends on host compiler version so that it does not report immediately.
> Hi! Thanks for the report! Do you have a link to the failed build or detailed error output and compile configuration?
Renamed the variable in 826e661a96a2aa7eb309dbca16c85a3d05edcec8


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122160



More information about the cfe-commits mailing list