On Fri, Feb 17, 2017 at 05:59:30PM +0100, Uros Bizjak wrote: > > Unfortunately this makes valgrind unhappy about that: > > https://bugzilla.redhat.com/show_bug.cgi?id=1423434 > > I assume it will complain now on anything pre-SSE2 that contains the memory > > barrier in 32-bit code. > > Perhaps we should decrement and increment %esp around it or something > > similar (or push/pop)? Of course, that would mean we need to take care > > of async unwind info. > > Or, we can simply revert the patch? Not that the barrier performance > of non-SSE 32bit targets matter...
Yeah. People who care about performance should use -m64 anyway. Jakub