[clang-tools-extra] r274501 - [include-fixer] reduce stack size by changing RegexHeaderMap to use const char * pair.

Eric Liu via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 4 06:34:12 PDT 2016


Author: ioeric
Date: Mon Jul  4 08:34:11 2016
New Revision: 274501

URL: http://llvm.org/viewvc/llvm-project?rev=274501&view=rev
Log:
[include-fixer] reduce stack size by changing RegexHeaderMap to use const char * pair.

Modified:
    clang-tools-extra/trunk/include-fixer/find-all-symbols/HeaderMapCollector.h
    clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp

Modified: clang-tools-extra/trunk/include-fixer/find-all-symbols/HeaderMapCollector.h
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/include-fixer/find-all-symbols/HeaderMapCollector.h?rev=274501&r1=274500&r2=274501&view=diff
==============================================================================
--- clang-tools-extra/trunk/include-fixer/find-all-symbols/HeaderMapCollector.h (original)
+++ clang-tools-extra/trunk/include-fixer/find-all-symbols/HeaderMapCollector.h Mon Jul  4 08:34:11 2016
@@ -23,8 +23,7 @@ namespace find_all_symbols {
 class HeaderMapCollector {
 public:
   typedef llvm::StringMap<std::string> HeaderMap;
-  typedef std::vector<std::pair<llvm::StringRef, llvm::StringRef>>
-      RegexHeaderMap;
+  typedef std::vector<std::pair<const char *, const char *>> RegexHeaderMap;
 
   HeaderMapCollector() : RegexHeaderMappingTable(nullptr) {}
 

Modified: clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp?rev=274501&r1=274500&r2=274501&view=diff
==============================================================================
--- clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp (original)
+++ clang-tools-extra/trunk/unittests/include-fixer/find-all-symbols/FindAllSymbolsTests.cpp Mon Jul  4 08:34:11 2016
@@ -74,7 +74,7 @@ public:
     std::string IncHeaderCode = "class IncHeaderClass {};";
 
     HeaderMapCollector::RegexHeaderMap RegexMap = {
-        {R"(internal_.*\.h$)", TopHeader},
+        {R"(internal_.*\.h$)", TopHeader.c_str()},
     };
 
     std::string InternalCode =




More information about the cfe-commits mailing list