[PATCH] D89142: llvmbuildectomy

Sat Oct 10 02:47:15 PDT 2020

thieta added a comment.

In D89142#2322663 <https://reviews.llvm.org/D89142#2322663>, @mehdi_amini wrote:

> Wow pretty neat to see `llvmbuild` being removed!
>
> That said we are still depending on it (we generate Bazel build file from this and use this in many projects using MLIR like TensorFlow, XLA, IREE...).
> We need to move away from it: can you ping me and/or @GMNGeoffrey when you get closer to have this in a landing state?

Could the generation of Bazel files be done by parsing the compile_commands.json maybe? that seems like a more "portable" approach since it's a pretty standard format these days.

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

https://reviews.llvm.org/D89142