[PATCH] Add tests for non-virtual call checking.

Peter Collingbourne peter at pcc.me.uk
Wed Apr 1 17:36:31 PDT 2015


================
Comment at: test/cfi/nvcall.cpp:51
@@ +50,3 @@
+  break_optimization(new Deriver<B, 2>);
+#endif
+
----------------
jfb wrote:
> Could you add a comment which explain the point of B32/B64/BM? IIUC it's just creating bigger numbers of derived classes, all with a bunch of virtuals?
Most of the tests in this directory use these macros, so I've documented them in a README.txt file in this directory.

http://reviews.llvm.org/D8792

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






More information about the llvm-commits mailing list