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

[James Henderson via llvm-commits]

================
@@ -113,12 +113,20 @@ static void strings(raw_ostream &OS, StringRef FileName,
     }
   };
 
-  // llvm-strings should be able to process a very large file on a
-  // memory-budgeted machine, so the file is read in chunks. To handle this, we
-  // read the file in chunk instead of copying the whole file into memory.
+  // To handle very large files without consuming excessive memory, we read the
+  // file in a little at a time and process it then rather than reading the
+  // entire file at once.
+  //
+  // A string is only buffered until it is known to be long enough to print;
+  // from then on it is streamed out directly, so an arbitrarily long string
+  // never needs an arbitrarily large buffer. Candidate therefore only ever
+  // holds a run that is shorter than MinLength and that was cut off by the end
+  // of a chunk.
+  const size_t Min = MinLength;
----------------
jh7370 wrote:

Nit: I'm seeing a mixture of `size_t` and `std::size_t`. I'm uncertain which is more common in LLVM without looking, but I imagine `size_t` would be preferred. Ultimately, use the same throughout the change.

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