[PATCH] D142024: [clang] Optimize clang::Builtin::Info density

2023-01-23 Thread serge via Phabricator via cfe-commits
serge-sans-paille added a comment. This got fixed in d1f313e73a1b33d662dbab54be76720b9c875645 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D142024/new/

[PATCH] D142024: [clang] Optimize clang::Builtin::Info density

2023-01-23 Thread Michael Buch via Phabricator via cfe-commits
Michael137 added a comment. Seems like this broke the LLDB incremental build: https://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/50184/execution/node/43/log/?consoleFull We build with `LLVM_ENABLE_MODULES` by default While building module 'Clang_Basic' imported from

[PATCH] D142024: [clang] Optimize clang::Builtin::Info density

2023-01-20 Thread Nikita Popov via Phabricator via cfe-commits
nikic accepted this revision. nikic added a comment. This revision is now accepted and ready to land. LGTM. The task-clock improvement is likely noise, but there's definitely a code size / max-rss win here. Comment at: clang/include/clang/Basic/BuiltinHeaders.def:9 +// +//