'Twas brillig, and Arun Raghavan at 28/09/11 09:08 did gyre and gimble: > On Wed, 2011-09-28 at 09:50 +0200, Maarten Bosmans wrote: >> This patch was sent to the list before, but was deemend to riscy pre-1.0. >> So now I'll resend. It does touch de same code as David fixed with 42bcb418aed SSE/MMX: Fix problem with highpitched >> noise on i386, but I leaves his solution intact, so it should be safe. >> >> The problem this patch is trying to solve is that the (pa_reg_x86) is not meaningfull with the "m" modifier. GCC ignores >> this, but clang (correctly) whines about it. > > Do we have an easy way to make sure the problem David fixed continues to > not be there with this patch? I was able to reproduce it pretty easily IIRC, so I should be able to test. Col -- Colin Guthrie gmane(at)colin.guthr.ie http://colin.guthr.ie/ Day Job: Tribalogic Limited http://www.tribalogic.net/ Open Source: Mageia Contributor http://www.mageia.org/ PulseAudio Hacker http://www.pulseaudio.org/ Trac Hacker http://trac.edgewall.org/
