[PATCH] D108274: Simplify setting up LLVM as bazel external repo

Geoffrey Martin-Noble via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 18 17:51:02 PDT 2021


GMNGeoffrey added a comment.

Huh I guess this works. To me the labels having `//utils/bazel` in them is pretty confusing because the WORKSPACE file is under utils/bazel already... Then again, making the way this loads the llvm-project repository consistent with how a dependency would load it is nice I guess. Let me make sure that this works for dependent projects (I missed some subtleties on the last one)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108274



More information about the llvm-commits mailing list