[PATCH] D38306: Cleanup some problems with LLVM_ENABLE_DUMP in release builds.

Don Hinton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 26 23:33:41 PDT 2017


hintonda created this revision.
Herald added subscribers: javed.absar, mgorny.

Cleanup some problems with LLVM_ENABLE_DUMP in release builds

Removed a few unneeded tests for NDEBUG and replace others with
#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP).

One issue involved ifdef'ing away a char* field from a struct, and changing 
tablegen to handle it correctly, which might not be the best way to handle it.


https://reviews.llvm.org/D38306

Files:
  CMakeLists.txt
  include/llvm/CodeGen/MachineRegisterInfo.h
  include/llvm/CodeGen/MachineScheduler.h
  include/llvm/CodeGen/TargetSchedule.h
  include/llvm/MC/MCSchedule.h
  lib/CodeGen/LiveDebugVariables.cpp
  lib/CodeGen/ScoreboardHazardRecognizer.cpp
  utils/TableGen/SubtargetEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38306.116766.patch
Type: text/x-patch
Size: 4479 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170927/f374840e/attachment.bin>


More information about the llvm-commits mailing list