[PATCH] D151273: [llvm-debuginfod][NFC] Switch to OptTable

Andres Villegas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 25 16:44:26 PDT 2023


avillega added a comment.

In D151273#4374565 <https://reviews.llvm.org/D151273#4374565>, @abrachet wrote:

> It would also be nice to update the bazel build. Technically we (because we don't work in google3) aren't required to keep the bazel build clean, though Google LLVM integrators will often see a commit was broken by a @ google.com address and ask that you fix it.
>
> In these cases, it might not be immediately obvious to the integrator how to fix this issues so it would certainly be courteous to fix it before landing. You can opt to join the bazel-build group https://reviews.llvm.org/project/profile/107/ which will add precommit checks to ensure the bazel build built cleanly.
>
> Though feel free to ignore

I couldn't find usages of llvm-debuginfod in the bazel files, I am assuming we currently do not build llvm-debuginfod with bazel given that it is not part of the default targets.



================
Comment at: llvm/tools/llvm-debuginfod/CMakeLists.txt:13
+  DEPENDS
+  GSYMUtilOptsTableGen
   )
----------------
abrachet wrote:
> 
You are very right, I copy pasted it from another 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151273



More information about the llvm-commits mailing list