commit f60650427d7b04042a05fd906fe3ac60b9c1757b
author Marcel Hlopko <hlopko@google.com> Mon Jul 01 00:17:11 2024 -0700
committer Copybara-Service <copybara-worker@google.com> Mon Jul 01 00:18:34 2024 -0700
tree b1755124374756779321d6b2c3bfee86b1b38aee
parent 9ec82cc8eb98dcff5e6691c07d4af5efc96da556

Rollback of:

Disable Crubit on grpc and targets that disable header parsing.

This allows Crubit to work on targets that depend on grpc, as long as they do not transitively include the headers. So, that's a workaround, at least.

This is not "strictly correct": a target might disable header parsing because it has a bad header in `srcs`, which isn't relevant to Crubit which only reads `hdrs`. However, it's almost certainly (also) broken in `srcs`. If ever we need to enable Crubit on a target that...

PiperOrigin-RevId: 648267674
Change-Id: I35cbdeed7928cfe93b133221b7201ddacb2bf813

bazel/llvm.bzl

diff --git a/bazel/llvm.bzl b/bazel/llvm.bzl
index b7c5c03..8f9a19a 100644
--- a/bazel/llvm.bzl
+++ b/bazel/llvm.bzl
@@ -53,7 +53,7 @@
             executable = False,
         )
 
-LLVM_COMMIT_SHA = "bb973785c9df1bf191ac4ff06ab1e7355dbd95c6"
+LLVM_COMMIT_SHA = "4713bd4ccc0c0d568f92916e7851d993291742c0"
 
 def llvm_loader_repository_dependencies():
     # This *declares* the dependency, but it won't actually be *downloaded* unless it's used.