llvmbot wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: Kazu Hirata (kazutakahirata) <details> <summary>Changes</summary> We don't need const on a return type. --- Full diff: https://github.com/llvm/llvm-project/pull/146382.diff 1 Files Affected: - (modified) clang/include/clang/Basic/TargetInfo.h (+1-1) ``````````diff diff --git a/clang/include/clang/Basic/TargetInfo.h b/clang/include/clang/Basic/TargetInfo.h index c3bce6e807f34..5c9031cc69dbb 100644 --- a/clang/include/clang/Basic/TargetInfo.h +++ b/clang/include/clang/Basic/TargetInfo.h @@ -1851,7 +1851,7 @@ class TargetInfo : public TransferrableTargetInfo, /// Returns the version of the darwin target variant SDK which was used during /// the compilation if one was specified, or an empty version otherwise. - const std::optional<VersionTuple> getDarwinTargetVariantSDKVersion() const { + std::optional<VersionTuple> getDarwinTargetVariantSDKVersion() const { return !getTargetOpts().DarwinTargetVariantSDKVersion.empty() ? getTargetOpts().DarwinTargetVariantSDKVersion : std::optional<VersionTuple>(); `````````` </details> https://github.com/llvm/llvm-project/pull/146382 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
