On 2014-05-08 12:17:51 +0300, Martin Storsjö wrote:
> The -MD option (that specifies using a dynamically linked CRT)
> can be interpreted by cpp to create a bogus dependency file named
> '-.d'.
> ---
> gas-preprocessor.pl | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/gas-preprocessor.pl b/gas-preprocessor.pl
> index f4a55a9..5216621 100755
> --- a/gas-preprocessor.pl
> +++ b/gas-preprocessor.pl
> @@ -150,6 +150,7 @@ if ($as_type ne "armasm") {
> @preprocess_c_cmd = grep ! /^-fp/, @preprocess_c_cmd;
> @preprocess_c_cmd = grep ! /^-EHsc$/, @preprocess_c_cmd;
> @preprocess_c_cmd = grep ! /^-O/, @preprocess_c_cmd;
> + @preprocess_c_cmd = grep ! /^-M(D|T)d?$/, @preprocess_c_cmd;
Where is this regexp pattern coming from? Are that the possible crt linking
options MSVC accepts in CFLAGS?
I would have expected only '/^-MD$/ to be removed since that is a valid
armsasm option which can't be supported while using it from
gas-preprocessor.pl.
Making the cmd line options compatible with gas is in scope but I not sure
whether ignoring unkown options just because libav's configure adds without
checking if they are supported by the assember is a good idea. I tend to
think it should fail like gas on unknow options.
I can look at the configure changes.
Janne
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
