[PATCH] D50901: [clang][ubsan] Split Implicit Integer Truncation Sanitizer into unsigned and signed checks
Roman Lebedev via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 10 23:18:51 PDT 2018
lebedev.ri added a comment.
In https://reviews.llvm.org/D50901#1261203, @rsmith wrote:
> This looks good to me. Sounds like @filcab is intending on doing another round of review too, so it'd make sense to double-check there before committing.
In https://reviews.llvm.org/D50901#1261312, @filcab wrote:
> LGTM on the clang side too.
>
> Thank you,
>
> Filipe
YAY \0/
Thank you for the review!
Repository:
rC Clang
https://reviews.llvm.org/D50901
More information about the cfe-commits
mailing list