From b6a56b8ef26a6b612eb5f49d37024666b073481e Mon Sep 17 00:00:00 2001 From: Tobias Stadler Date: Thu, 12 Jun 2025 15:50:31 +0100 Subject: [PATCH] [llvm-remarkutil] bitstream2yaml: Keep output file (#143220) Keep the output file on successful exit, otherwise `llvm-remarkutil bitstream2yaml -o filename.yaml ...` does not produce any output, because the output file is deleted when the tool exits. --- llvm/test/tools/llvm-remarkutil/convert.test | 7 ++++--- llvm/tools/llvm-remarkutil/RemarkConvert.cpp | 1 + 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/llvm/test/tools/llvm-remarkutil/convert.test b/llvm/test/tools/llvm-remarkutil/convert.test index 83023c8ce6a8..0d7ab8e4682a 100644 --- a/llvm/test/tools/llvm-remarkutil/convert.test +++ b/llvm/test/tools/llvm-remarkutil/convert.test @@ -1,6 +1,7 @@ -RUN: llvm-remarkutil bitstream2yaml %p/Inputs/two-remarks.bitstream -o - | FileCheck %s -strict-whitespace -RUN: llvm-remarkutil yaml2bitstream %p/Inputs/two-remarks.yaml -o %t -RUN: llvm-remarkutil bitstream2yaml %t -o - | FileCheck %s -strict-whitespace +RUN: llvm-remarkutil bitstream2yaml %p/Inputs/two-remarks.bitstream -o %t.yaml +RUN: FileCheck %s -strict-whitespace < %t.yaml +RUN: llvm-remarkutil yaml2bitstream %p/Inputs/two-remarks.yaml -o %t.bitstream +RUN: llvm-remarkutil bitstream2yaml %t.bitstream -o - | FileCheck %s -strict-whitespace ; CHECK: --- !Analysis ; CHECK-NEXT: Pass: prologepilog diff --git a/llvm/tools/llvm-remarkutil/RemarkConvert.cpp b/llvm/tools/llvm-remarkutil/RemarkConvert.cpp index 35d8dcd99b4a..207c5e0a8048 100644 --- a/llvm/tools/llvm-remarkutil/RemarkConvert.cpp +++ b/llvm/tools/llvm-remarkutil/RemarkConvert.cpp @@ -133,6 +133,7 @@ static Error tryBitstream2YAML() { if (!E.isA()) return E; consumeError(std::move(E)); + OF->keep(); return Error::success(); } } // namespace bitstream2yaml