On Wed, 12 May 2021 19:15:48 +0200 Thomas Huth <th...@redhat.com> wrote:
> Clang versions before v11.0 insist on having the %rX or %cX register > names instead of just a number. Since our Travis-CI is currently > still using Clang v6.0, we have to fix this to avoid failing jobs. > > Signed-off-by: Thomas Huth <th...@redhat.com> > --- > pc-bios/s390-ccw/helper.h | 2 +- > pc-bios/s390-ccw/jump2ipl.c | 4 ++-- > pc-bios/s390-ccw/menu.c | 8 ++++---- > pc-bios/s390-ccw/virtio.c | 2 +- > 4 files changed, 8 insertions(+), 8 deletions(-) > > diff --git a/pc-bios/s390-ccw/helper.h b/pc-bios/s390-ccw/helper.h > index dfcfea0ff0..3d0731c4c6 100644 > --- a/pc-bios/s390-ccw/helper.h > +++ b/pc-bios/s390-ccw/helper.h > @@ -31,7 +31,7 @@ static inline void *u32toptr(uint32_t n) > > static inline void yield(void) > { > - asm volatile ("diag 0,0,0x44" > + asm volatile ("diag %%r0,%%r0,0x44" > : : > : "memory", "cc"); > } Sigh, this really looks uglier, but if it pleases the compiler... Reviewed-by: Cornelia Huck <coh...@redhat.com>