delcypher added a comment. In D97327#2585410 <https://reviews.llvm.org/D97327#2585410>, @jansvoboda11 wrote:
> LGTM. > > If you don't get around committing this today, please rebase this on top of > D97375 <https://reviews.llvm.org/D97375> that I'm going to commit tomorrow. @jansvoboda11 This doesn't build anymore because https://reviews.llvm.org/D97375 removed `AutoNormalizeEnum`. Based on skim reading the patch it looks like I should remove the `AutoNormalizeEnum` mixin and replace `MarshallingInfoString` with `MarshallingInfoEnum`. Does that sound right? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97327/new/ https://reviews.llvm.org/D97327 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
