[PATCH] D125725: [SystemZ][z/OS] Add PPA1 support

Wed May 18 12:51:59 PDT 2022

yusra.syeda added a comment.

In D125725#3523243 <https://reviews.llvm.org/D125725#3523243>, @mib wrote:

> Hey @yusra.syeda ! It looks like your patch broke the macOS lldb bot: https://green.lab.llvm.org/green/job/lldb-cmake/
>
>   /Users/buildslave/jenkins/workspace/lldb-cmake/llvm-project/llvm/include/llvm/BinaryFormat/GOFF.h:23:23: error: missing '#include <_types/_uint8_t.h>'; 'uint8_t' must be declared before it is used
>   enum SubsectionKind : uint8_t {
>                         ^
>   /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.15.sdk/usr/include/_types/_uint8_t.h:31:23: note: declaration here is not visible
>   typedef unsigned char uint8_t;
>
> You're basically missing an include. Let me know if you need help reproducing / fixing this 🙂

Thanks for pointing that out. I posted a patch to fix this here: https://reviews.llvm.org/D125921

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D125725