[PATCH] D133890: [CMake] Do these replacements to make use of D132608

Saleem Abdulrasool via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 19 10:58:20 PDT 2022


compnerd added a comment.

I don't see anything wrong with this change per se, but I'm conflicted on the name of the variable.  These are not standard variables but are encroaching on the CMake namespace.  What happens if upstream decides to use these names?  I think that we should keep the names in the `LLVM_` namespace.  However, I realize that the variable itself is not being touched here and this is a mechanical replacement.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D133890



More information about the cfe-commits mailing list