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

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to