[PATCH] D48164: [llvm-exegesis] Print the whole snippet in analysis.

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 14 08:48:14 PDT 2018


gchatelet accepted this revision.
gchatelet added inline comments.
This revision is now accepted and ready to land.


================
Comment at: tools/llvm-exegesis/lib/Analysis.cpp:94
+                            const char *Separator) const {
+  llvm::SmallVector<std::string, 3> Lines;
+  // Parse the asm snippet and print it.
----------------
gchatelet wrote:
> Why not a `llvm::SmallVector<llvm::SmallString<16>, 3>` ?
Ping?


Repository:
  rL LLVM

https://reviews.llvm.org/D48164





More information about the llvm-commits mailing list