On 2014-08-08 14:58:39 +0200, Diego Biurrun wrote:
> On Thu, Aug 07, 2014 at 06:39:33PM +0200, Diego Biurrun wrote:
> > On Wed, Aug 06, 2014 at 04:41:24PM +0200, Janne Grunau wrote:
> > > On 2014-08-06 05:57:39 -0700, Diego Biurrun wrote:
> > > 
> > > the commit message describes the commit only partially. It also uses 
> > > gas-preprocessor.pl if it's available and not only on darwin. I think 
> > > that's not going to work since the plain 'gas-preprocessor.pl' without 
> > > options will produce mach-O specific code.
> > 
> > So should we instead try plain "gas" before combining it with the
> > gas-preprocessor?
> 
> I assume you mean one of
> 
> -as-type apple-gas, -as-type apple-clang, -as-type armasm

Also plain clang and plain gas although I think gas is not really used.
--arch should be set too

> How do you suggest we choose between them?  Set the required options in
> the OS and other sections?

we set $as_type which might have all required information. The 
distinction between apple-gas and apple-clang might be not correct but I 
don't think it matters in practice.

case $as_type
  arm*) as_type=armasm ;;
  gcc)  as_type=gas ;;
esac
[ $target_os = "darwin" ] && as_type="apple-$as_type"

not nice but should work

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

Reply via email to