Author: Aditya Sinha Date: 2026-11-02T10:37:22+05:30 New Revision: aa0a14e24b420b6f21a16b358b56b21739ddf936
URL: https://github.com/llvm/llvm-project/commit/aa0a14e24b420b6f21a16b358b56b21739ddf936 DIFF: https://github.com/llvm/llvm-project/commit/aa0a14e24b420b6f21a16b358b56b21739ddf936.diff LOG: [NFC][clang][Driver] Changed Triple constructor (#236542) This PR changes the constructor used to build the triple in the `OffloadTargetInfo` constructor. It builds the `Triple` directly from the joined string, instead of parsing it into a temporary and then making a second `Triple` from its components. This avoids creating the same object twice. This is an NFC change. Added: Modified: clang/lib/Driver/OffloadBundler.cpp Removed: ################################################################################ diff --git a/clang/lib/Driver/OffloadBundler.cpp b/clang/lib/Driver/OffloadBundler.cpp index 2d0c326ab6f40..1989bc6631fde 100644 --- a/clang/lib/Driver/OffloadBundler.cpp +++ b/clang/lib/Driver/OffloadBundler.cpp @@ -90,9 +90,7 @@ OffloadTargetInfo::OffloadTargetInfo(const StringRef Target, this->OffloadKind = Components.front(); ArrayRef<StringRef> TripleSlice{&Components[1], /*length=*/4}; - llvm::Triple T = llvm::Triple(llvm::join(TripleSlice, "-")); - this->Triple = llvm::Triple(T.getArchName(), T.getVendorName(), T.getOSName(), - T.getEnvironmentName()); + this->Triple = llvm::Triple(llvm::join(TripleSlice, "-")); } bool OffloadTargetInfo::hasHostKind() const { _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
