[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