[llvm] [Support] mmap when possible in getSTDIN. (PR #162013)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 7 00:49:33 PDT 2025


https://github.com/jh7370 commented:

I've given this a bit of thought and while there may be some use to the change you're trying to make for other tools, I actually think it's not the right solution for llvm-strings, since it doesn't fix the general case of reading from stdin. I actually think the correct thing to do would be to change llvm-strings to not try to read in the whole file up front, but rather read it in as required (presumably using some kind of stream implementation or similar). This will require changes to the core logic of the program, but it will reduce the memory usage for files as well as stdin.

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


More information about the llvm-commits mailing list