Should there be some test coverage for this change (or was this fixing
an already failing test)?

~Aaron

On Thu, Apr 14, 2022 at 2:01 AM Fangrui Song via cfe-commits
<cfe-commits@lists.llvm.org> wrote:
>
>
> Author: Fangrui Song
> Date: 2022-04-13T23:00:57-07:00
> New Revision: 26dbb93704bf39a198909d04ad8c49b4bde46cce
>
> URL: 
> https://github.com/llvm/llvm-project/commit/26dbb93704bf39a198909d04ad8c49b4bde46cce
> DIFF: 
> https://github.com/llvm/llvm-project/commit/26dbb93704bf39a198909d04ad8c49b4bde46cce.diff
>
> LOG: [Driver] Fix -fpascal-strings on Darwin
>
> Added:
>
>
> Modified:
>     clang/lib/Driver/ToolChains/Darwin.cpp
>
> Removed:
>
>
>
> ################################################################################
> diff  --git a/clang/lib/Driver/ToolChains/Darwin.cpp 
> b/clang/lib/Driver/ToolChains/Darwin.cpp
> index 9f0eeea8d54e2..aaa4c6e5aabb8 100644
> --- a/clang/lib/Driver/ToolChains/Darwin.cpp
> +++ b/clang/lib/Driver/ToolChains/Darwin.cpp
> @@ -2554,11 +2554,11 @@ DerivedArgList *MachO::TranslateArgs(const 
> DerivedArgList &Args,
>        break;
>
>      case options::OPT_fpascal_strings:
> -      DAL->AddFlagArg(A, Opts.getOption(options::OPT_mpascal_strings));
> +      DAL->AddFlagArg(A, Opts.getOption(options::OPT_fpascal_strings));
>        break;
>
>      case options::OPT_fno_pascal_strings:
> -      DAL->AddFlagArg(A, Opts.getOption(options::OPT_mno_pascal_strings));
> +      DAL->AddFlagArg(A, Opts.getOption(options::OPT_fno_pascal_strings));
>        break;
>      }
>    }
>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits@lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to