[PATCH] D116046: [mlir] Add `mlir/unittests/BUILD.bazel`

Geoffrey Martin-Noble via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 20 11:20:20 PST 2021


GMNGeoffrey added inline comments.


================
Comment at: utils/bazel/llvm-project-overlay/mlir/unittests/BUILD.bazel:1
+# Description:
+#   Unit tests for MLIR.
----------------
Please add the license header as in other build files. Also, the "Description:" seems not very useful and the description itself honestly doesn't say anything that the filepath doesn't. I'd just delete both


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116046



More information about the llvm-commits mailing list