[llvm] [llvm-strings] Use small buffer instead of reading whole file (PR #163073)

[via llvm-commits]

================
@@ -110,19 +113,37 @@ static void strings(raw_ostream &OS, StringRef FileName, StringRef Contents) {
     OS << L << '\n';
   };
 
-  const char *B = Contents.begin();
-  const char *P = nullptr, *E = nullptr, *S = nullptr;
-  for (P = Contents.begin(), E = Contents.end(); P < E; ++P) {
-    if (isPrint(*P) || *P == '\t') {
-      if (S == nullptr)
-        S = P;
-    } else if (S) {
-      print(S - B, StringRef(S, P - S));
-      S = nullptr;
+  Buffer.resize_for_overwrite(sys::fs::DefaultReadChunkSize);
+  std::string StringBuffer;
----------------
aokblast wrote:

I think it increases the runtime in some cases as it adds some branch to test the buffer. But I switch in accordance with your suggestion now.

https://github.com/llvm/llvm-project/pull/163073