alexfh added a comment.

In https://reviews.llvm.org/D31130#707197, @aaron.ballman wrote:

> 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.


I've made the test script add `-nostdinc++` in r298501, which should help 
catching this kind of an issue earlier.


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