[llvm] [llvm-readobj][ELF] Remove flush from `GNUELFDumper::printField()` (PR #217602)

James Henderson via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 20 06:07:54 PDT 2026


https://github.com/jh7370 commented:

Seems reasonable in principle. Two questions:
1) Should we remove the other explicit flushes in the file too (I imagine they're not as problematic because they aren't called as often, but if they're not doing anything useful...)
2) Can this result in error messages in the middle of a line of regular output, if stdout and stderr are piped to the same file? I know at one point efforts were made to prevent this from happening, with additional flushing added somewhere, but I suspect it was elsewhere and so this won't cause any problems with that.

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


More information about the llvm-commits mailing list