c-rhodes added a comment. In D109489#2991261 <https://reviews.llvm.org/D109489#2991261>, @jansvoboda11 wrote:
> LGTM. > > To give more background, I think my intention was to add an assertion that > `name` and `other_name` are the same (except for the `no_` part) and I was > waiting for TableGen assertions to land. But I don't think that's too > valuable, since the callers already ensure that by some other mechanism. This > should be fine to remove. No worries thanks for clarifying! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109489/new/ https://reviews.llvm.org/D109489 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits