[PATCH] D48958: [clang][ubsan] Implicit Cast Sanitizer - integer truncation - clang part

Roman Lebedev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jul 26 04:19:20 PDT 2018


lebedev.ri updated this revision to Diff 157458.
lebedev.ri marked an inline comment as done.
lebedev.ri added a reviewer: erichkeane.
lebedev.ri added a subscriber: erichkeane.
lebedev.ri added a comment.

Address @rsmith & @erichkeane [IRC] review notes:

- https://reviews.llvm.org/D49838 - [AST] Sink 'part of explicit cast' down into ImplicitCastExpr
- https://reviews.llvm.org/D49844 - [AST] Add a isActuallyImplicitCast() helper to the CastExpr class.
- Drop no longer needed `CastExprStackGuard`, `ScalarExprEmitter::IsTopCastPartOfExplictCast()`, just use `CastExpr::isActuallyImplicitCast()` directly.

This should be a NFC change, there should not be any functionality change because of this.


Repository:
  rC Clang

https://reviews.llvm.org/D48958

Files:
  docs/ReleaseNotes.rst
  docs/UndefinedBehaviorSanitizer.rst
  include/clang/Basic/Sanitizers.def
  include/clang/Basic/Sanitizers.h
  lib/CodeGen/CGExprScalar.cpp
  lib/CodeGen/CodeGenFunction.h
  lib/Driver/SanitizerArgs.cpp
  lib/Driver/ToolChain.cpp
  test/CodeGen/catch-implicit-integer-truncations.c
  test/CodeGenCXX/catch-implicit-integer-truncations.cpp
  test/Driver/fsanitize.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D48958.157458.patch
Type: text/x-patch
Size: 45413 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20180726/af1c31a5/attachment-0001.bin>


More information about the cfe-commits mailing list