On Sun, 5 Aug 2018 14:28:25 -0400 Pavel Zbitskiy <pavel.zbits...@gmail.com> wrote:
Meta note #1: please add the s390x maintainers on cc: for the cover letter as well; that makes tracking easier Meta note #2: I don't think this really fits trivial? I'll be happy to take the end result via the s390x tree, anyway. > Add BAL, BALR, CVB instructions > Fix few bugs in PACK, CSST > > Pavel Zbitskiy (6): > target/s390x: add BAL and BALR instructions > target/s390x: fix CSST decoding and runtime alignment check > target/s390x: fix ipm polluting irrelevant bits > target/s390x: add EX support for TRT and TRTR > target/s390x: fix PACK reading 1 byte less and writing 1 byte more > target/s390x: implement CVB, CVBY and CVBG > > target/s390x/helper.h | 2 ++ > target/s390x/insn-data.def | 7 ++++++ > target/s390x/int_helper.c | 58 > ++++++++++++++++++++++++++++++++++++++++++++++ > target/s390x/mem_helper.c | 24 +++++++++++++++---- > target/s390x/translate.c | 52 +++++++++++++++++++++++++++++++++++++---- > 5 files changed, 135 insertions(+), 8 deletions(-) > Out of curiousity: Do you have some code that actually makes use of these instructions (especially BAL/BALR), or have you found this by looking at the documentation?