erichkeane added a comment. I don't see much value in these changes. In the cases I know anything about, the change in state (or out params) is the error checking that we need. The casts add nothing but noise, so I don't think these are good changes.
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147791/new/ https://reviews.llvm.org/D147791 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
