This is the mail archive of the mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH, rs6000] Emit correct note for DWARF CFI information on LE prolog VSX stores

On Sat, Nov 16, 2013 at 10:32 PM, Bill Schmidt
<> wrote:
> Hi,
> For VSX in little endian we currently split vector register stores into
> a permute/store pair.  For prolog stores, this results in a
> REG_FRAME_RELATED_EXPR note that doesn't have a simple register for its
> RHS, which it needs to have.  This patch detects that situation and
> ensures we produce the correct note.
> This problem was breaking bootstrap when configured with
> --with-cpu=power7, something we hadn't tried before.  With the patch we
> now get past stage 1.  There is at least one wrong-code bug to track
> down in stage 2, but modifying this note is clearly not involved with
> that.
> Otherwise bootstrapped and tested on powerpc64-unknown-linux-gnu with no
> regressions on the big-endian side, also bootstrapped with
> --with-cpu=power7.  Is this ok for trunk?
> Thanks,
> Bill
> 2011-11-16  Bill Schmidt  <>
>         * config/rs6000/rs6000.c (rs6000_frame_related): Add split_reg
>         parameter and use it in REG_FRAME_RELATED_EXPR note.
>         (emit_frame_save): Call rs6000_frame_related with extra NULL_RTX
>         parameter.
>         (rs6000_emit_prologue): Likewise, but for little endian VSX
>         stores, pass the source register of the store instead.


Thanks, David

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]