[PATCH] D58297: [Sema] SequenceChecker: C++17 sequencing rules for built-in operators <<, >>, .*, ->*, =, op=

Bruno Ricci via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 15 14:26:02 PST 2019


riccibruno updated this revision to Diff 187089.
riccibruno added a comment.

1. Only call `note{Pre,Post}{Use, Mod}` if we have a valid memory location.
2. Fixed 2 tests in `CXX/drs` which I initially missed.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58297/new/

https://reviews.llvm.org/D58297

Files:
  lib/Sema/SemaChecking.cpp
  test/CXX/drs/dr2xx.cpp
  test/CXX/drs/dr6xx.cpp
  test/SemaCXX/warn-unsequenced.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D58297.187089.patch
Type: text/x-patch
Size: 17230 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190215/19460f3a/attachment-0001.bin>


More information about the cfe-commits mailing list