[PATCH] D135436: [Bindings] Remove Go bindings

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 7 15:28:31 PDT 2022


MaskRay accepted this revision.
MaskRay added a comment.
Herald added a subscriber: StephenFan.

Looks good to me. The Go tests often caused failures when a new attribute was introduced and contributors usually don't know how to deal with them. They were also flaky in the past so was disabled as the resolution to https://github.com/llvm/llvm-project/issues/53017 .
I agree that out-of-tree bindings may be more useful and the maintainers can decide support more than one versions.


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

https://reviews.llvm.org/D135436



More information about the llvm-commits mailing list