[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