diff options
| author | Uros Bizjak <ubizjak@gmail.com> | 2013-11-11 21:02:19 +0100 |
|---|---|---|
| committer | Uros Bizjak <uros@gcc.gnu.org> | 2013-11-11 21:02:19 +0100 |
| commit | 5783ad0ea7403304e4ea9c06818373a3bc7e62b1 (patch) | |
| tree | 751c66d387964afc5ded5eb70600325d74f8b33e /gcc/config/i386/i386.c | |
| parent | eb47f18904bc0ec0556a217beb0ae15a4a2d9b11 (diff) | |
| download | gcc-5783ad0ea7403304e4ea9c06818373a3bc7e62b1.zip gcc-5783ad0ea7403304e4ea9c06818373a3bc7e62b1.tar.gz gcc-5783ad0ea7403304e4ea9c06818373a3bc7e62b1.tar.bz2 | |
re PR target/58853 (ICE in expand_set_or_movmem_prologue_epilogue_by_misaligned_moves)
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:
PR target/58853
* gcc.target/i386/pr58853.c: New test.
Co-Authored-By: H.J. Lu <hongjiu.lu@intel.com>
From-SVN: r204685
Diffstat (limited to 'gcc/config/i386/i386.c')
| -rw-r--r-- | gcc/config/i386/i386.c | 16 |
1 files changed, 9 insertions, 7 deletions
diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c index 430d562..8b8cdfae 100644 --- a/gcc/config/i386/i386.c +++ b/gcc/config/i386/i386.c @@ -23761,13 +23761,15 @@ ix86_expand_set_or_movmem (rtx dst, rtx src, rtx count_exp, rtx val_exp, } gcc_assert (desired_align >= 1 && align >= 1); - /* Misaligned move sequences handles both prologues and epilogues at once. - Default code generation results in smaller code for large alignments and - also avoids redundant job when sizes are known precisely. */ - misaligned_prologue_used = (TARGET_MISALIGNED_MOVE_STRING_PROLOGUES - && MAX (desired_align, epilogue_size_needed) <= 32 - && ((desired_align > align && !align_bytes) - || (!count && epilogue_size_needed > 1))); + /* Misaligned move sequences handle both prologue and epilogue at once. + Default code generation results in a smaller code for large alignments + and also avoids redundant job when sizes are known precisely. */ + misaligned_prologue_used + = (TARGET_MISALIGNED_MOVE_STRING_PRO_EPILOGUES + && MAX (desired_align, epilogue_size_needed) <= 32 + && desired_align <= epilogue_size_needed + && ((desired_align > align && !align_bytes) + || (!count && epilogue_size_needed > 1))); /* Do the cheap promotion to allow better CSE across the main loop and epilogue (ie one load of the big constant in the |