[PATCH] D50901: [clang][ubsan] Split Implicit Integer Truncation Sanitizer into unsigned and signed checks

[Richard Smith - zygoloid via Phabricator via cfe-commits]

rsmith accepted this revision.
rsmith added a comment.
This revision is now accepted and ready to land.

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.



================
Comment at: docs/UndefinedBehaviorSanitizer.rst:101
+     conversions - when either one, or both of the types are signed.
      Issues caught by this sanitizer are not undefined behavior,
      but are often unintentional.
----------------
this sanitizer -> these sanitizers


Repository:
  rC Clang

https://reviews.llvm.org/D50901