[PATCH] D141855: [include-mapping] Parse zombie_names.html into a removed symbols map.

[Haojian Wu via Phabricator via cfe-commits]

hokein added a comment.

Have you run the script on the zombie_names page? Does it work?



================
Comment at: clang/tools/include-mapping/gen_std.py:67
                       required=True)
+  parser.add_argument('-output',
+                      default='SymbolMap.inc',
----------------
instead adding two CLI flags, I'd suggest extending the existing `language` flag to something like `-symbols = {cpp_symbols, c_symbols, cpp_removed_symbols}`, it is easy to extend in the future (e.g. `c_removed_symbols`).



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141855