[PATCH] D117176: [bazel] Separate capi_deps from deps in mlir_c_api_cc_library.

[Geoffrey Martin-Noble via Phabricator via llvm-commits]

GMNGeoffrey added inline comments.


================
Comment at: utils/bazel/llvm-project-overlay/mlir/build_defs.bzl:34
         header_deps = [],
+        capi_deps = [],
         **kwargs):
----------------
GMNGeoffrey wrote:
> stellaraccident wrote:
> > GMNGeoffrey wrote:
> > > Can you document this argument? I think we're getting in to the territory of non-obvious on this one. I would probably just bite the bullet and do the standard "Args:" block, but mostly because I'm used to linters complaining if I don't do exactly that. I would say that header_deps and capi_deps are worth documenting here, at least.
> > How's that?
> 👍 
{meme, src="dont-cross-the-streams"}




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117176