reames added a comment. In D133443#3801899 <https://reviews.llvm.org/D133443#3801899>, @asb wrote:
> It looks like they're still missing in this updated version of the patch? I have no idea what's going wrong here. I had been very careful to make sure the patch contained the new test file, but you're right, the revision in phab didn't. Since we now had two LGTMs, and the tests had been in the original patch upload, I went ahead and landed. The tests are in the committed patch; if you want any changes, let me know. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133443/new/ https://reviews.llvm.org/D133443 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits