delcypher accepted this revision. delcypher added a comment. This revision is now accepted and ready to land.
@yaxunl Thanks for addressing my feedback so quickly. I think the commit message should also mention that `KEYCUDA` is now included in `KEYALL`. Other than that LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125396/new/ https://reviews.llvm.org/D125396 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits