[PATCH] D151022: [llvm-exegesis] Introduce SubprocessMemory Utility Class

Mon Jun 26 18:10:50 PDT 2023

aidengrossman added a comment.

In D151022#4450888 <https://reviews.llvm.org/D151022#4450888>, @Jake-Egan wrote:

> I'm not sure if it's this commit or one of the other llvm-exegesis commits you pushed, but there's a build failure on AIX now:
>
>   /scratch/powerllvm/powerllvm_env/aix-ppc64/clang-ppc64-aix/llvm-project/llvm/tools/llvm-exegesis/lib/X86/Target.cpp:49:33: error: unused variable 'VAddressSpaceCeiling' [-Werror,-Wunused-const-variable]
>   static constexpr const intptr_t VAddressSpaceCeiling = 0x0000800000000000;
>
> https://lab.llvm.org/buildbot/#/builders/214/builds/8173/steps/5/logs/stdio
>
> Could you take look please?

Looks like since AIX isn't Linux, `__linux__` doesn't get defined and none of the new functions that use that variable end up getting compiled. "Should be a simple" `#ifdef`. I'll get a patch up soon that will hopefully fix the issue.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151022/new/

https://reviews.llvm.org/D151022