This is an archive of the discontinued LLVM Phabricator instance.

[SystemZ][z/OS] Add PPA1 support
ClosedPublic

Authored by yusra.syeda on May 16 2022, 1:37 PM.

Details

Summary

This patch adds the PPA1 fields to the assembly file for z/OS.

Diff Detail

Event Timeline

yusra.syeda created this revision.May 16 2022, 1:37 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 16 2022, 1:37 PM
Herald added a subscriber: hiraditya. · View Herald Transcript
yusra.syeda requested review of this revision.May 16 2022, 1:37 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 16 2022, 1:37 PM
uweigand accepted this revision.May 17 2022, 3:15 AM

LGTM, thanks!

This revision is now accepted and ready to land.May 17 2022, 3:15 AM

apply clang format

Kai accepted this revision.May 18 2022, 9:50 AM

LGTM.

This revision was landed with ongoing or failed builds.May 18 2022, 11:13 AM
This revision was automatically updated to reflect the committed changes.
mib added a subscriber: mib.EditedMay 18 2022, 12:06 PM

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 🙂

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