[PATCH] D136392: [Bazel] Use `LLVM_VERSION` from `llvm/CMakeLists.txt`

[Jordan Rupprecht via Phabricator via llvm-commits]

rupprecht added a comment.

This looks like a nice improvement! Just one thing I noticed.



================
Comment at: utils/bazel/configure.bzl:143
 
+    vars = _extract_cmake_settings(
+        repository_ctx,
----------------
I see a warning that `vars` is unused here and should be removed. Did you mean to have `_extract_cmake_settings()` only parse the information, and have a separate helper method that actually writes out the file, i.e. split the previous method at `# Write out individual vars`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D136392