[PATCH] D80803: [DebugInfo] Add flush to ensure debug line errors are in right place

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 1 08:00:39 PDT 2020


jhenderson added a comment.

Having investigated further, updating the defaultWarningHandler method to flush stdout does not solve the issue, because llvm-dwarfdump actually writes to a file stream, which means the thing being flushed is not the right stream. I think the right thing to do is to have llvm-dwarfdump provide its own warning and error handler to DWARFContext. That will allow it to flush the right stream at the right time.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D80803/new/

https://reviews.llvm.org/D80803





More information about the llvm-commits mailing list