dang marked 2 inline comments as done. dang added inline comments.
================ Comment at: clang/include/clang/Driver/Options.td:3455 + HelpText<"Specify target triple (e.g. i686-apple-darwin9)">, + MarshallingInfoString<"TargetOpts->Triple", "llvm::sys::getDefaultTargetTriple()", "std::string">, + AlwaysEmit, Normalizer<"normalizeTriple">, DenormalizeString; ---------------- dang wrote: > herhut wrote: > > There is some explicit normalization missing here. In CC1Options.td this is > > > > ``` > > def triple : Separate<["-"], "triple">, > > HelpText<"Specify target triple (e.g. i686-apple-darwin9)">, > > MarshallingInfoString<"TargetOpts->Triple", > > "llvm::Triple::normalize(llvm::sys::getDefaultTargetTriple())", > > "std::string">, > > AlwaysEmit, Normalizer<"normalizeTriple">, DenormalizeString; > > ``` > > > > It seems the normalizer does not apply to the defaults and we now see a > > failure in `clang/unittests/Frontend/CompilerInvocationTest.cpp` for > > powerpc targets. > Of course, I must have lost it at some point. Can you link me to the failure > for reference. Done in 0555db0a5df4 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82574/new/ https://reviews.llvm.org/D82574 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
