[llvm] [SystemZ][z/OS] fix sample-split-layout.test on z/OS (PR #109679)

Zibi Sarbinowski via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 23 08:58:40 PDT 2024


https://github.com/zibi2 created https://github.com/llvm/llvm-project/pull/109679

The Lit in subject fails on z/OS since the input file `Inputs/split-layout.profdata` is marked as ASCII even though it is binary and the output file `Output/sample-split-layout.test.tmp-output` is binary. 

This PR removes the diff command which fails because it compares a binary file and a text file. The rational is that this diff command seems to be redundant to the `FileCheck` on the next command.

>From d600297860f8352ac7b5514c6a94b97bd6e51b69 Mon Sep 17 00:00:00 2001
From: Zbigniew Sarbinowski <zibi at ca.ibm.com>
Date: Mon, 23 Sep 2024 15:57:56 +0000
Subject: [PATCH] fix sample-split-layout.test on z/OS

---
 llvm/test/tools/llvm-profdata/sample-split-layout.test | 1 -
 1 file changed, 1 deletion(-)

diff --git a/llvm/test/tools/llvm-profdata/sample-split-layout.test b/llvm/test/tools/llvm-profdata/sample-split-layout.test
index a1927c80b2e29b..51091e9f25c02d 100644
--- a/llvm/test/tools/llvm-profdata/sample-split-layout.test
+++ b/llvm/test/tools/llvm-profdata/sample-split-layout.test
@@ -1,5 +1,4 @@
 RUN: llvm-profdata merge --sample --extbinary --split-layout %p/Inputs/sample-profile.proftext -o %t-output
-RUN: diff %t-output %p/Inputs/split-layout.profdata
 
 RUN: llvm-profdata merge --sample --text --split-layout %t-output | FileCheck %s
 CHECK: main:184019:0



More information about the llvm-commits mailing list