[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