[PATCH] Omit 'nodebug' method from the class

Paul Robinson Paul_Robinson at playstation.sony.com
Thu Jun 25 10:45:44 PDT 2015


================
Comment at: test/CodeGenCXX/debug-info-method-nodebug.cpp:6
@@ +5,3 @@
+  void absent() __attribute__((nodebug));
+  int i;
+};
----------------
dblaikie wrote:
> do you need the int member?
No.  I'll take it out.
(Leftover from a fatter test that also checked codegen didn't emit any dbg annotations in a definition of absent().  But it turned out there's already CodeGen/attr-nodebug.c which is testing the codegen part, so I removed the definition.)

Hmmm looks like attr-nodebug.c could be more robust... I'll look at that in a bit.

http://reviews.llvm.org/D10678

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






More information about the cfe-commits mailing list