As a quick note, this broke the test because you didn't update the line numbers in the checks.
Fixed thusly: echristo@athyra ~/r/llvm-project> git push To github.com:llvm/llvm-project.git 6bed43f3c40..259a9b10390 master -> master :) -eric On Thu, Dec 12, 2019 at 12:25 PM Petr Hosek via Phabricator via cfe-commits <cfe-commits@lists.llvm.org> wrote: > phosek accepted this revision. > phosek added a comment. > This revision is now accepted and ready to land. > > LGTM > > > CHANGES SINCE LAST ACTION > https://reviews.llvm.org/D71419/new/ > > https://reviews.llvm.org/D71419 > > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits