This is the mail archive of the gcc-patches@gcc.gnu.org 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]

[PATCH, i38]: Fix PR58853, ICE in expand_set_or_movmem_prologue_epilogue_by_misaligned_moves


Hello!

Attached patch fixes a typo in ix86_expand_set_or_movmem, where a
wrong define was used in a condition.  The patch also adds additional
condition (as proposed by H.J.) as a correctness improvement (although
patched gcc bootstraps and regression tests OK without).

I took the liberty to rename a couple of (long) constants and to fix
some typos while there.

2013-11-11  Uros Bizjak  <ubizjak@gmail.com>
        H.J. Lu  <hongjiu.lu@intel.com>

    PR target/58853
    * config/i386/x86-tune.def
    (X86_TUNE_MISALIGNED_MOVE_STRING_PRO_EPILOGUES): Rename from
    TARGET_MISALIGNED_MOVE_STRING_PROLOGUES.
    * config/i386/i386.h
    (TARGET_MISALIGNED_MOVE_STRING_PRO_EPILOGUES): Rename from
    TARGET_MISALIGNED_MOVE_STRING_PROLOGUES_EPILOGUES.  Update for renamed
    X86_TUNE_MISALIGNED_MOVE_STRING_PRO_EPILOGUES.
    * config/i386/i386.c (ix86_expand_set_or_movmem): Use
    TARGET_MISALIGNED_MOVE_STRING_PRO_EPILOGUES to calculate
    misaligned_prologue_used.  Check that
    desired_aling <= epilogue_size_needed.

testsuite/ChangeLog:

2013-11-11  Uros Bizjak  <ubizjak@gmail.com>

    PR target/58853
    * gcc.target/i386/pr58853.c: New test.

Patch was approved by Honza in the PR.

Patch was bootstrapped and regression tested on x86_64-pc-linux-gnu
{,-m32} and committed to mainline SVN.

Uros.

Attachment: p.diff.txt
Description: Text document


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