sammccall added a comment. The Bracket struct & translation between token/bracket streams is not really justified by the implementation in this patch. Next patch will run nontrivial algorithms on the bracket strings.
Happy to scope this patch down (avoid that struct) or up (include the full algorithm) if you prefer, but I think this keeps the diffs clearest. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125911/new/ https://reviews.llvm.org/D125911 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits