[PATCH] D14047: [MC] Get rid of NDEBUG in headers

David Majnemer via llvm-commits llvm-commits at lists.llvm.org
Sun Oct 25 10:59:37 PDT 2015


majnemer added a subscriber: majnemer.

================
Comment at: llvm/CodeGen/TargetSchedule.h:110-115
@@ -109,9 +109,8 @@
 
-#ifndef NDEBUG
+  /// \brief Get the name of the resource (used for debugging purposes).
   const char *getResourceName(unsigned PIdx) const {
     if (!PIdx)
       return "MOps";
     return SchedModel.getProcResource(PIdx)->Name;
   }
 
----------------
This shouldn't have ABI impact, should it?  Why not leave it in the NDEBUG?


http://reviews.llvm.org/D14047





More information about the llvm-commits mailing list