[PATCH] D158557: [clang] Fix crash in __builtin_strncmp and other related builtin functions

[Erich Keane via Phabricator via cfe-commits]

erichkeane added a comment.

Are there any Sema tests we can add to show that we warn/diagnose/SOMETHING on 
these?  If someone passes a negative size, we should probably at least do the 
warning that it was converted/truncated.



================
Comment at: clang/lib/AST/ExprConstant.cpp:9357
       APSInt N;
       if (!EvaluateInteger(E->getArg(2), N, Info))
----------------
Context missing throughout


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

https://reviews.llvm.org/D158557

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits