This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug target/55147] [4.8 Regression] x86: wrong code for 64-bit load
- From: "ubizjak at gmail dot com" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: Thu, 01 Nov 2012 09:48:50 +0000
- Subject: [Bug target/55147] [4.8 Regression] x86: wrong code for 64-bit load
- Auto-submitted: auto-generated
- References: <bug-55147-4@http.gcc.gnu.org/bugzilla/>
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55147
--- Comment #6 from Uros Bizjak <ubizjak at gmail dot com> 2012-11-01 09:48:50 UTC ---
(In reply to comment #5)
> Created attachment 28589 [details]
> gcc48-pr55147.patch
>
> So like this? Or do you want to merge the bswap{si,di}2 expanders using SWI48
> iterator too? That would make i386.md tiny bit shorter, but would make
> gen_bswapdi2 longer (as the compiler can't figure out that for DImode (thus
> TARGET_64BIT) TARGET_BSWAP is always true). Perhaps I could do
> else if (<MODE>mode == DImode || TARGET_BSWAP)
> so that at least optimized gcc builds would optimize it away.
No, your proposed patch is OK and pre-approved for mainline SVN.