[PATCH] Add test suite for the Control Flow Integrity feature.

Peter Collingbourne peter at pcc.me.uk
Thu Feb 19 16:35:56 PST 2015


================
Comment at: test/cfi/anon-namespace.cpp:43
@@ +42,3 @@
+
+#endif
+
----------------
kcc wrote:
> #endif  // TU1
> here and below
Done

================
Comment at: test/cfi/multiple-inheritance.cpp:35
@@ +34,3 @@
+  puts("1");
+  fflush(stdout);
+
----------------
kcc wrote:
> If you use stderr you will not need fflush
Done

http://reviews.llvm.org/D7738

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list