This patch adds the PPA1 fields to the assembly file for z/OS.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
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 🙂
Comment Actions
Thanks for pointing that out. I posted a patch to fix this here: https://reviews.llvm.org/D125921