warning during build on solaris-sparc

[Paul Hohensee]

yes, your change should make the warning go away.

btw, while you're changing that, you could replace the 'ba 5f'
with 'retl'.

Paul

Keith McGuigan wrote:
> Hello,
>
> I think I've seen this compilation warning for a while but never 
> really cared enough to look at it.  But now I'm using a different tool 
> to go through compiler errors and I'm running into it over and over 
> again, so I was wondering if it was possible/worth fixing (or already 
> fixed or already has a CR for it).
>
> It's this warning from solaris_sparc.s:
>
> ../hotspot/src/os_cpu/solaris_sparc/vm/solaris_sparc.s|84| warning: 
> label in delay slot (follows CTI)
>
> This is the code that causes it:
>
> -------------
>  _Copy_conjoint_jlongs_atomic:
>          cmp     %o0, %o1
>          bleu    4f
>          sll     %o2, 3, %o4
>          ba      2f
>     1:
>          subcc   %o4, 8, %o4
>          std     %o2, [%o1]
> -------------
>
> I suppose this type of change would make the warning go away (?):
> -------------
>  _Copy_conjoint_jlongs_atomic:
>          cmp     %o0, %o1
>          bleu    4f
>          sll     %o2, 3, %o4
>          ba      2f
> +        subcc   %o4, 8, %o4
>     1:
>          subcc   %o4, 8, %o4
>          std     %o2, [%o1]
> -------------
>
> Does that look like a reasonable fix?  Does anyone know if a CR 
> already exists for this?
>