[pulseaudio-discuss] [PATCH] Make pulse build with clang again

[Colin Guthrie]

'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/