[PATCH] D72212: [Sema] Improve -Wrange-loop-analysis warnings

Dávid Bolvanský via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Jan 4 15:45:40 PST 2020


xbolva00 added inline comments.


================
Comment at: clang/lib/Sema/SemaStmt.cpp:2806
-  // constructors.
-  if (VariableType.isPODType(SemaRef.Context))
     return;
----------------
Mordante wrote:
> xbolva00 wrote:
> > assert it?
> Sorry I don't understand what you mean. What do you want to assert?
if (isTriviallyCopyableType) {
     assert(isPOD);
}

But probably not very useful to have an assert here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72212





More information about the cfe-commits mailing list