[aarch64-port-dev ] RFR(XS): 8211207: AArch64: Fix build failure after JDK-8211029
Pengfei Li (Arm Technology China)
Pengfei.Li at arm.com
Thu Sep 27 08:42:48 UTC 2018
I'm happy to hear Andrew Haley will post a better fix. You can discard my change.
--
Thanks,
Pengfei
> -----Original Message-----
>
> On 09/27/2018 10:21 AM, Andrew Haley wrote:
> > On 09/27/2018 08:22 AM, Aleksey Shipilev wrote:
> >> Otherwise looks good and trivial.
> >
> > No, it is neither good nor trivial:some of those should be
> ShouldNotReachHere().
> > I'll post a webrev later.
>
> Okay, let's see it.
>
> I was mostly concerned with having the same control flow as before, did I
> miss some change that is actually non-trivial? On the second read, this
> change in c1_LIRAssembler_aarch64.cpp looks suspicious, as it elevates
> ShouldNotReachHere to default case, rather than letting default thing fall-
> through?
>
> break;
> + default:
> ShouldNotReachHere();
> }
>
>
> -Aleksey
>
More information about the build-dev
mailing list