[FFmpeg-devel] [PATCH] avfilter/vf_fspp.c: make the filter compile with latest LLVM

[Michael Niedermayer]

On Thu, Oct 31, 2013 at 02:33:47PM -0700, Chandler Carruth wrote:
> On Thu, Oct 31, 2013 at 1:43 PM, Thierry Foucu <tfoucu at gmail.com> wrote:
> 
> > /home/michael/ffmpeg-git/ffmpeg/libavfilter/libmpcodecs/vf_fspp.c:878:9:
> >> error: ran out of registers during register allocation
> >>         ".p2align " "4" "\n\t"
> >>         ^
> >> /home/michael/ffmpeg-git/ffmpeg/libavfilter/libmpcodecs/vf_fspp.c:878:9:
> >> error: ran out of registers during register allocation
> >> 3 warnings and 2 errors generated.
> >>
> >>
> >
> > Any idea how we can resolve this problem?
> >
> > Could it be a bug in clang then?
> >
> 
> This was a common Clang bug which is (AFAIK) now fixed.
> 

> I'm actually more surprised that the old version worked?

just tested, yes it seems working


> What code did it
> generate?

i think this one was which failed:

column_fidct_mmx:                       # @column_fidct_mmx
# BB#0:                                 # %entry
    pushl   %ebp
    movl    %esp, %ebp
    pushl   %edi
    pushl   %esi
    subl    $64, %esp
    movl    20(%ebp), %eax
    movl    16(%ebp), %ecx
    movl    12(%ebp), %edx
    movl    8(%ebp), %esi
    movl    %esi, -12(%ebp)
    movl    %edx, -16(%ebp)
    movl    %ecx, -20(%ebp)
    movl    %eax, -24(%ebp)
    movl    -16(%ebp), %eax
    movl    -20(%ebp), %ecx
    movl    -24(%ebp), %edx
    movl    -12(%ebp), %esi
    movl    %edx, -60(%ebp)         # 4-byte Spill
    movl    %esi, %edx
    movl    -60(%ebp), %esi         # 4-byte Reload
    movl    %esi, -64(%ebp)         # 4-byte Spill
    movl    %eax, %esi
    movl    -64(%ebp), %eax         # 4-byte Reload
    movl    %ecx, -68(%ebp)         # 4-byte Spill
    movl    %eax, %ecx
    movl    -68(%ebp), %edi         # 4-byte Reload
    #APP

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20131101/19438314/attachment.asc>