[PATCH] D137451: [CMake] Use LLVM_RUNTIME_TARGET in runtimes

Petr Hosek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 28 09:47:26 PDT 2023


phosek added a comment.

In D137451#4651440 <https://reviews.llvm.org/D137451#4651440>, @mstorsjo wrote:

> In D137451#4651362 <https://reviews.llvm.org/D137451#4651362>, @phosek wrote:
>
>> This should be addressed in the latest version.
>
> Ok, so this now retains `LLVM_DEFAULT_TARGET_TRIPLE` as the external user-settable cache variable that controls this, while the internal code is refactored to use `LLVM_RUNTIME_TARGET` instead? And I guess it's possible to build this by just setting `LLVM_RUNTIME_TARGET` directly too, even if that's not a cache variable?

Yes, that's correct.

> That seems reasonable to me. Would the next steps after that be to do an actual deprecation cycle - warning users if they're setting the old variable name and making `LLVM_RUNTIME_TARGET` a cache variable as well? And then after the next release (or later?), remove handling of the old variable name here?

That was my plan assuming we don't want to keep support for `LLVM_DEFAULT_TARGET_TRIPLE` which would be my preference.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137451/new/

https://reviews.llvm.org/D137451



More information about the llvm-commits mailing list