Re: [PATCH 4/8] s390/entry: workaround llvm's IAS limitations

From: Heiko Carstens
Date: Thu May 12 2022 - 15:16:47 EST


On Thu, May 12, 2022 at 12:06:59PM -0700, Nathan Chancellor wrote:
> > +#ifdef CONFIG_AS_IS_LLVM
> > + clgfrl %r14,.Lrange_size\@
> > +#else
> > + clgfi %r14,\end - \start
> > +#endif
> > jhe \outside_label
> > +#ifdef CONFIG_CC_IS_CLANG
>
> I think this one also wants to be CONFIG_AS_IS_LLVM, right?

Yes, of course.

> Other than that, seems fine to me, although I have no knowledge of s390
> assembly so that statement probably means next to nothing :)

Thanks for having a look! :)