[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