From e9f28133a9fb42156a76f03e8beb85be93b793d2 Mon Sep 17 00:00:00 2001 From: George Rimar Date: Sat, 17 Aug 2019 15:24:16 +0000 Subject: [PATCH] [llvm-readobj] - An attemp to fix BB after r369191. Few BB failed with the following error: Command Output (stderr): -- /home/buildbots/ppc64be-clang-lnt-test/clang-ppc64be-lnt/llvm/test/tools/llvm-readobj/stack-sizes.test:263:19: error: BADSECTION-OUT: expected string not found in input # BADSECTION-OUT: 8 ? ^ :4:1: note: scanning from here ^ It doesn't reproduce on ubuntu/windows I have. Also, seems many of the bots are happy too. This slightly reorders the code to make fouts().flush() call earlier, like it was before the r369191. llvm-svn: 369192 --- llvm/tools/llvm-readobj/llvm-readobj.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/llvm/tools/llvm-readobj/llvm-readobj.cpp b/llvm/tools/llvm-readobj/llvm-readobj.cpp index 8e89ccc..00f7ab2 100644 --- a/llvm/tools/llvm-readobj/llvm-readobj.cpp +++ b/llvm/tools/llvm-readobj/llvm-readobj.cpp @@ -395,14 +395,14 @@ LLVM_ATTRIBUTE_NORETURN void reportError(Error Err, StringRef Input) { void reportWarning(Error Err, StringRef Input) { assert(Err); + // Flush the standard output to print the warning at a + // proper place. + fouts().flush(); if (Input == "-") Input = ""; handleAllErrors(createFileError(Input, std::move(Err)), [&](const ErrorInfoBase &EI) { - // Flush the standard output to print the warning at a - // proper place. - fouts().flush(); errs() << "\n"; WithColor::warning(errs()) << EI.message() << "\n"; }); -- 2.7.4