[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
Wed Sep 27 17:18:57 PDT 2017


hintonda added inline comments.


================
Comment at: llvm/trunk/CMakeLists.txt:400
+if( LLVM_ENABLE_ASSERTIONS )
+  set(LLVM_ENABLE_DUMP ON)
+endif()
----------------
Just noticed that this doesn't actually update the cache.  However, that doesn't matter in this case since the updated variable is what is actually used.  

I'll fix this shortly so it won't confuse anyone looking at the cache.


Repository:
  rL LLVM

https://reviews.llvm.org/D38306





More information about the llvm-commits mailing list