bkramer added inline comments. ================ Comment at: include-fixer/find-all-symbols/STLPostfixHeaderMap.cpp:16 @@ -15,2 +15,3 @@ const HeaderMapCollector::HeaderMap *getSTLPostfixHeaderMap() { static const HeaderMapCollector::HeaderMap STLPostfixHeaderMap = { + {"include/__stddef_max_align_t.h$", "<cstddef>"}, ---------------- Can we make this a std::vector<std::pair<llvm::Regex, const char *>>? Creating llvm::Regex is somewhat expensive so we only want to do it once. We also never lookup into the postfix header map and only iterate over it so using a StringMap for it is wasteful.
http://reviews.llvm.org/D21787 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits