[llvm] Revert "[MemProf] Drop unneccessary REQUIRES: x86-linux directives." (PR #142816)

Snehasish Kumar via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 4 09:58:52 PDT 2025


https://github.com/snehasish created https://github.com/llvm/llvm-project/pull/142816

Reverts llvm/llvm-project#142718

Breaks ppc aix builds:     https://lab.llvm.org/buildbot/#/builders/64/builds/4036


>From 2907a0b0a15642c12c92d15a95c01cce191e9cf1 Mon Sep 17 00:00:00 2001
From: Snehasish Kumar <snehasishk at google.com>
Date: Wed, 4 Jun 2025 09:58:18 -0700
Subject: [PATCH] Revert "[MemProf] Drop unneccessary REQUIRES: x86-linux
 directives. (#142718)"

This reverts commit a87c4eef1d8ee69f46986985ec5cf35215992e5a.
---
 .../Transforms/PGOProfile/memprof-call-site-at-alloc-site.ll    | 1 +
 .../Transforms/PGOProfile/memprof-dump-matched-alloc-site.ll    | 1 +
 .../Transforms/PGOProfile/memprof-dump-matched-call-sites.ll    | 1 +
 llvm/test/Transforms/PGOProfile/memprof-undrift.test            | 1 +
 llvm/test/Transforms/PGOProfile/memprof_annotate_yaml.test      | 1 +
 llvm/test/Transforms/PGOProfile/memprof_discard_threshold.ll    | 2 ++
 .../Transforms/PGOProfile/memprof_match_hot_cold_new_calls.ll   | 2 ++
 llvm/test/Transforms/PGOProfile/memprof_missing_leaf.ll         | 2 ++
 llvm/test/tools/llvm-profdata/memprof-yaml-invalid.test         | 1 +
 9 files changed, 12 insertions(+)

diff --git a/llvm/test/Transforms/PGOProfile/memprof-call-site-at-alloc-site.ll b/llvm/test/Transforms/PGOProfile/memprof-call-site-at-alloc-site.ll
index a0a359cea6a36..b5a4648dad74d 100644
--- a/llvm/test/Transforms/PGOProfile/memprof-call-site-at-alloc-site.ll
+++ b/llvm/test/Transforms/PGOProfile/memprof-call-site-at-alloc-site.ll
@@ -2,6 +2,7 @@
 ; allocation call stack but does not call one of the memory allocation
 ; functions.
 
+; REQUIRES: x86_64-linux
 ; RUN: split-file %s %t
 ; RUN: llvm-profdata merge %t/memprof-call-site-at-alloc-site.yaml -o %t/memprof-call-site-at-alloc-site.memprofdata
 ; RUN: opt < %t/memprof-call-site-at-alloc-site.ll -passes='memprof-use<profile-filename=%t/memprof-call-site-at-alloc-site.memprofdata>' -memprof-print-match-info -S 2>&1 | FileCheck %s
diff --git a/llvm/test/Transforms/PGOProfile/memprof-dump-matched-alloc-site.ll b/llvm/test/Transforms/PGOProfile/memprof-dump-matched-alloc-site.ll
index 993cc42ff41a3..2dcaa9d492869 100644
--- a/llvm/test/Transforms/PGOProfile/memprof-dump-matched-alloc-site.ll
+++ b/llvm/test/Transforms/PGOProfile/memprof-dump-matched-alloc-site.ll
@@ -23,6 +23,7 @@
 ;
 ; Here we expect to match the allocation site to encompass 3 frames.
 
+; REQUIRES: x86_64-linux
 ; RUN: split-file %s %t
 ; RUN: llvm-profdata merge %t/memprof-dump-matched-alloc-site.yaml -o %t/memprof-dump-matched-alloc-site.memprofdata
 ; RUN: opt < %t/memprof-dump-matched-alloc-site.ll -passes='memprof-use<profile-filename=%t/memprof-dump-matched-alloc-site.memprofdata>' -memprof-print-match-info -S 2>&1 | FileCheck %s
diff --git a/llvm/test/Transforms/PGOProfile/memprof-dump-matched-call-sites.ll b/llvm/test/Transforms/PGOProfile/memprof-dump-matched-call-sites.ll
index ed3a079a93f73..fa99116b820f9 100644
--- a/llvm/test/Transforms/PGOProfile/memprof-dump-matched-call-sites.ll
+++ b/llvm/test/Transforms/PGOProfile/memprof-dump-matched-call-sites.ll
@@ -31,6 +31,7 @@
 ; Note that f3 is considered to be an allocation site, not a call site, because
 ; it directly calls new after inlining.
 
+; REQUIRES: x86_64-linux
 ; RUN: split-file %s %t
 ; RUN: llvm-profdata merge %t/memprof-dump-matched-call-site.yaml -o %t/memprof-dump-matched-call-site.memprofdata
 ; RUN: opt < %t/memprof-dump-matched-call-site.ll -passes='memprof-use<profile-filename=%t/memprof-dump-matched-call-site.memprofdata>' -memprof-print-match-info -S 2>&1 | FileCheck %s
diff --git a/llvm/test/Transforms/PGOProfile/memprof-undrift.test b/llvm/test/Transforms/PGOProfile/memprof-undrift.test
index b2006fa3f8bf8..5529c7701f6ec 100644
--- a/llvm/test/Transforms/PGOProfile/memprof-undrift.test
+++ b/llvm/test/Transforms/PGOProfile/memprof-undrift.test
@@ -1,3 +1,4 @@
+; REQUIRES: x86_64-linux
 
 ; Make sure that we can undrift the MemProf profile and annotate the IR
 ; accordingly.
diff --git a/llvm/test/Transforms/PGOProfile/memprof_annotate_yaml.test b/llvm/test/Transforms/PGOProfile/memprof_annotate_yaml.test
index 28ded32082b19..1ef963dec9e23 100644
--- a/llvm/test/Transforms/PGOProfile/memprof_annotate_yaml.test
+++ b/llvm/test/Transforms/PGOProfile/memprof_annotate_yaml.test
@@ -1,3 +1,4 @@
+; REQUIRES: x86_64-linux
 
 ; Make sure that we can ingest the MemProf profile in YAML and
 ; annotate a call to new as cold.
diff --git a/llvm/test/Transforms/PGOProfile/memprof_discard_threshold.ll b/llvm/test/Transforms/PGOProfile/memprof_discard_threshold.ll
index 92a2a0ecd789d..a1f8369243888 100644
--- a/llvm/test/Transforms/PGOProfile/memprof_discard_threshold.ll
+++ b/llvm/test/Transforms/PGOProfile/memprof_discard_threshold.ll
@@ -1,5 +1,7 @@
 ;; Tests option to discard small noncold contexts.
 
+;; Avoid failures on big-endian systems that can't read the profile properly
+; REQUIRES: x86_64-linux
 
 ;; Generate the profile and the IR.
 ; RUN: split-file %s %t
diff --git a/llvm/test/Transforms/PGOProfile/memprof_match_hot_cold_new_calls.ll b/llvm/test/Transforms/PGOProfile/memprof_match_hot_cold_new_calls.ll
index 596323c662deb..4aa0511622662 100644
--- a/llvm/test/Transforms/PGOProfile/memprof_match_hot_cold_new_calls.ll
+++ b/llvm/test/Transforms/PGOProfile/memprof_match_hot_cold_new_calls.ll
@@ -1,6 +1,8 @@
 ;; Tests optional matching of memprof profile on call to operator new
 ;; with manual hot/cold hint.
 
+;; Avoid failures on big-endian systems that can't read the profile properly
+; REQUIRES: x86_64-linux
 
 ;; Generate the profile and the IR.
 ; RUN: split-file %s %t
diff --git a/llvm/test/Transforms/PGOProfile/memprof_missing_leaf.ll b/llvm/test/Transforms/PGOProfile/memprof_missing_leaf.ll
index 3c1c58d62647e..e6fb1863acbce 100644
--- a/llvm/test/Transforms/PGOProfile/memprof_missing_leaf.ll
+++ b/llvm/test/Transforms/PGOProfile/memprof_missing_leaf.ll
@@ -3,6 +3,8 @@
 ;; matching and we are able to match the next call frame up the inlined
 ;; context.
 
+;; Avoid failures on big-endian systems that can't read the profile properly
+; REQUIRES: x86_64-linux
 
 ;; # To generate below LLVM IR for use in matching.
 ;; $ clang++ -gmlt -fdebug-info-for-profiling -S memprof_missing_leaf.cc \
diff --git a/llvm/test/tools/llvm-profdata/memprof-yaml-invalid.test b/llvm/test/tools/llvm-profdata/memprof-yaml-invalid.test
index 0810bcdaa40cc..a13451cee5116 100644
--- a/llvm/test/tools/llvm-profdata/memprof-yaml-invalid.test
+++ b/llvm/test/tools/llvm-profdata/memprof-yaml-invalid.test
@@ -1,3 +1,4 @@
+; REQUIRES: x86_64-linux
 ; RUN: split-file %s %t
 ; RUN: not llvm-profdata merge %t/memprof-invalid.yaml -o %t/memprof-invalid.indexed
 



More information about the llvm-commits mailing list