teemperor marked 3 inline comments as done.
teemperor added inline comments.
================
Comment at: test/CXX/except/except.spec/libc-empty-except.cpp:6
+#include "libc-empty-except.h"
+
+void f() {
----------------
bruno wrote:
> In a testcase like this but using the actual real headers, if you do:
>
> #include <xmmintrin.h>
> #include <stdlib.h>
>
> does it also compiles fine?
(I already answered this offline back then, but for the record again): Yes, we
tried this actual code and I think the test case (with the actual reverse fix
applied) also should cover this case.
================
Comment at: test/CXX/except/except.spec/libc-empty-except.cpp:27
+#else
+#include "mm_malloc.h"
+#include "stdlib.h"
----------------
v.g.vassilev wrote:
> Maybe we should reverse the includes here as we discussed offline.
Thanks!
https://reviews.llvm.org/D43871
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
