-
Notifications
You must be signed in to change notification settings - Fork 11.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[SystemZ][z/OS] fix sample-split-layout.test on z/OS #109679
Conversation
@llvm/pr-subscribers-pgo Author: Zibi Sarbinowski (zibi2) ChangesThe Lit in subject fails on z/OS since the input file 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 Full diff: https://github.com/llvm/llvm-project/pull/109679.diff 1 Files Affected:
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
|
ping |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
LLVM Buildbot has detected a new failure on builder Full details are available at: https://lab.llvm.org/buildbot/#/builders/70/builds/3729 Here is the relevant piece of the build log for the reference
|
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.
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 fileOutput/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.