[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