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

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 27 17:57:30 PDT 2017


MatzeB added inline comments.


================
Comment at: llvm/trunk/CMakeLists.txt:400
+if( LLVM_ENABLE_ASSERTIONS )
+  set(LLVM_ENABLE_DUMP ON)
+endif()
----------------
hintonda wrote:
> 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.
Yep it does not update the cache and that is a good thing in this situation IMO... I'd rather not see cmake change setting B just because I switched setting A...


Repository:
  rL LLVM

https://reviews.llvm.org/D38306





More information about the llvm-commits mailing list