[clang] [clang][Darwin] Simply deployment version assignment in the Driver (PR #142013)

Steven Wu via cfe-commits cfe-commits at lists.llvm.org
Thu May 29 13:02:24 PDT 2025


================
@@ -2313,12 +2340,12 @@ void Darwin::AddDeploymentTarget(DerivedArgList &Args) const {
   SDKInfo = parseSDKSettings(getVFS(), Args, getDriver());
 
   // The OS and the version can be specified using the -target argument.
-  std::optional<DarwinPlatform> OSTarget =
+  std::optional<DarwinPlatform> DP =
----------------
cachemeifyoucan wrote:

I feel `DP` is not a good variable name as it is used everywhere but it is hard to tell what it is from variable name.

https://github.com/llvm/llvm-project/pull/142013


More information about the cfe-commits mailing list