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

Reply via email to