[PATCH] D29839: [clang-tidy] New misc-istream-overflow check

Kostya Kortchinsky via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 10 13:17:26 PST 2017


cryptoad marked 5 inline comments as done.
cryptoad added inline comments.


================
Comment at: clang-tidy/misc/IstreamOverflowCheck.cpp:78-80
+    if (!Arg->isIntegerConstantExpr(WidthSize, Context)) {
+      llvm::errs() << "Couldn't get width() size.\n";
+    }
----------------
Prazek wrote:
> debug?
Oops!
I'd like to somehow keep that in there (for debug purposes). Any preferred way> (ifdef DEBUG or other).


Repository:
  rL LLVM

https://reviews.llvm.org/D29839





More information about the cfe-commits mailing list