[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