[PATCH] D153937: [llvm-exegesis] Exclude loongarch when define GLIBC_INITS_RSEQ
Aiden Grossman via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 27 21:19:57 PDT 2023
aidengrossman accepted this revision.
aidengrossman added a comment.
This revision is now accepted and ready to land.
One nit, otherwise LGTM.
================
Comment at: llvm/tools/llvm-exegesis/lib/BenchmarkRunner.cpp:42
#ifdef __GLIBC__
-#if __GLIBC_MINOR__ >= 35
+#if __GLIBC_MINOR__ >= 35 && !defined(__loongarch__)
#define GLIBC_INITS_RSEQ
----------------
Do you mind adding a comment around here explaining why loongarch is excluded (similar to the review description)?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153937/new/
https://reviews.llvm.org/D153937
More information about the llvm-commits
mailing list