[PATCH] D35960: [MachineOutliner] Cleanup: move findCandidates out of suffix tree

Jessica Paquette via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 27 16:09:24 PDT 2017


paquette marked an inline comment as done.
paquette added inline comments.


================
Comment at: lib/CodeGen/MachineOutliner.cpp:1054-1055
   std::ostringstream NameStream;
-  NameStream << "OUTLINED_FUNCTION" << "_" << OF.Name;
+  NameStream << "OUTLINED_FUNCTION"
+             << "_" << OF.Name;
 
----------------
MatzeB wrote:
> can't you just make this `"OUTLINE_FUNCTION_"`?
I think that sounds a little strange, since the function was "outlined"... But it's a character shorter so I'll go ahead and change it.


https://reviews.llvm.org/D35960





More information about the llvm-commits mailing list