On 2014-06-02 14:49:39 +0200, Diego Biurrun wrote:
> On Fri, May 30, 2014 at 08:56:30PM +0200, Janne Grunau wrote:
> > Clang's integrated assembler (after 3.4) does not yet support the
> > '.altmacro' directive which is only used in arm asm. Support is planned:
> > http://llvm.org/bugs/show_bug.cgi?id=18918
> > ---
> >  configure | 6 ++++++
> >  1 file changed, 6 insertions(+)
> > 
> > --- a/configure
> > +++ b/configure
> > @@ -3888,6 +3888,12 @@ if enabled asm; then
> >  .endm
> >  m x
> >  EOF
> > +    if enabled arm; then
> > +        check_as <<EOF && enable gnu_as || \
> > +        $nogas "GNU assembler not found, install gas-preprocessor"
> > +.altmacro
> > +EOF
> > +    fi
> 
> You could move this up into the ARM section where I believe it would fit
> better.

I tend to disagree. If moved to arm section I would have to wrap it into
if enabled asm ; then ...; fi.

I could remove the 'if enabled arm;' here though without breaking anything 
though. It would at most print spurious warnings on ppc.

Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to