[PATCH] D46910: [Support] Avoid normalization in sys::getDefaultTargetTriple

[Reid Kleckner via Phabricator via llvm-commits]

rnk added a comment.

Applying this diff to clang fixes the test failures:

  $ git diff ../clang
  diff --git a/clang/lib/Frontend/CompilerInvocation.cpp b/clang/lib/Frontend/CompilerInvocation.cpp
  index 05e5196e32d2..4824bdbba581 100644
  --- a/clang/lib/Frontend/CompilerInvocation.cpp
  +++ b/clang/lib/Frontend/CompilerInvocation.cpp
  @@ -2916,10 +2916,11 @@ static void ParseTargetArgs(TargetOptions &Opts, ArgList &Args,
     Opts.FPMath = Args.getLastArgValue(OPT_mfpmath);
     Opts.FeaturesAsWritten = Args.getAllArgValues(OPT_target_feature);
     Opts.LinkerVersion = Args.getLastArgValue(OPT_target_linker_version);
  -  Opts.Triple = llvm::Triple::normalize(Args.getLastArgValue(OPT_triple));
  +  Opts.Triple = Args.getLastArgValue(OPT_triple);
     // Use the default target triple if unspecified.
     if (Opts.Triple.empty())
       Opts.Triple = llvm::sys::getDefaultTargetTriple();
  +  Opts.Triple = llvm::Triple::normalize(Opts.Triple);
     Opts.OpenCLExtensionsAsWritten = Args.getAllArgValues(OPT_cl_ext_EQ);
     Opts.ForceEnableInt128 = Args.hasArg(OPT_fforce_enable_int128);
     Opts.NVPTXUseShortPointers = Args.hasFlag(


Repository:
  rL LLVM

https://reviews.llvm.org/D46910