aaron.ballman reopened this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

This change was reverted in r298470. The use of the <assert.h> include is a 
problem because this is not a clang-supplied header file. Also, there's a 
(good) question about what array decay is happening in the `assert(false)` test 
case.


https://reviews.llvm.org/D31130



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D31130: ... Breno Rodrigues Guimaraes via Phabricator via cfe-commits
    • [PATCH] D31... Aaron Ballman via Phabricator via cfe-commits
    • [PATCH] D31... Breno Rodrigues Guimaraes via Phabricator via cfe-commits
    • [PATCH] D31... Aaron Ballman via Phabricator via cfe-commits
    • [PATCH] D31... Breno Rodrigues Guimaraes via Phabricator via cfe-commits
    • [PATCH] D31... Aaron Ballman via Phabricator via cfe-commits
    • [PATCH] D31... Breno Rodrigues Guimaraes via Phabricator via cfe-commits
    • [PATCH] D31... Aaron Ballman via Phabricator via cfe-commits
    • [PATCH] D31... Aaron Ballman via Phabricator via cfe-commits
    • [PATCH] D31... Alexander Kornienko via Phabricator via cfe-commits
    • [PATCH] D31... Breno Rodrigues Guimaraes via Phabricator via cfe-commits
    • [PATCH] D31... Matthias Gehre via Phabricator via cfe-commits

Reply via email to