aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land.
Oh, great catch! Thank you for the fix. The changes here LGTM, but precommit CI failed because of patch application. That issue was resolved on the precommit CI machine, so it might not be a bad idea to push up a rebased version of this patch to run precommit CI before landing just to be sure there aren't surprises, or be sure to land it when you're around to react to build failures if any do crop up. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128373/new/ https://reviews.llvm.org/D128373 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits