IIRC GCC can optimize expressions like "(a<<8)|(a>>8)", but has problems with "(a<<8)+(a>>8)". I don't remember where I've read this though. This patch changes '+' to '|' in ENDSWAP_16 and ENDSWAP_32 macros. -------------- next part -------------- A non-text attachment was scrubbed... Name: endswap.patch Type: application/octet-stream Size: 631 bytes Desc: not available Url : http://lists.xiph.org/pipermail/flac-dev/attachments/20140807/bc7e788c/attachment.obj
lvqcl wrote:> IIRC GCC can optimize expressions like "(a<<8)|(a>>8)", but has problems > with "(a<<8)+(a>>8)". I don't remember where I've read this though. > > This patch changes '+' to '|' in ENDSWAP_16 and ENDSWAP_32 macros.Applied. Thanks. Erik -- ---------------------------------------------------------------------- Erik de Castro Lopo http://www.mega-nerd.com/