[PATCH] D119884: [llvm] [bindings/OCaml] Remove unused dep on ounit2

Shiwei Weng via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 16 08:07:29 PST 2022


arbipher accepted this revision.
arbipher added a comment.

It looks good to me.

My 5 cent (for ocaml) is to keep it and it would be easier to really add some useful ounit tests, however, this PR also has shown what needs to do to revert it back.


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

https://reviews.llvm.org/D119884



More information about the llvm-commits mailing list