[PATCH] D127459: [mlir][NFC] Rename Bazel target aliases and consolidate targets

Geoffrey Martin-Noble via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 9 17:04:47 PDT 2022


GMNGeoffrey added a comment.

In D127459#3571908 <https://reviews.llvm.org/D127459#3571908>, @Mogball wrote:

> Yes, I'll be taking at look at the CMake names afterwards. This one goes after the low-hanging fruit, mostly, in the bazel targets. I'll probably do a comprehensive review to make sure everything lines up in bazel and CMake

Sounds great, thank you. I had been thinking it would make sense to only have to change the Bazel names once, but I'm certainly not going to complain about it :-)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127459



More information about the llvm-commits mailing list