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.