[llvm] r340561 - Fixup AreCoreFilesPrevented() to consider first LLVM_ENABLE_CRASH_DUMPS and secondly coreFilesPrevented.

[Douglas Yung via llvm-commits]

Author: dyung
Date: Thu Aug 23 11:20:13 2018
New Revision: 340561

URL: http://llvm.org/viewvc/llvm-project?rev=340561&view=rev
Log:
Fixup AreCoreFilesPrevented() to consider first LLVM_ENABLE_CRASH_DUMPS and secondly coreFilesPrevented.

The previous change ignored the latter resulting in crash dumps being generated when LLVM_ENABLE_CRASH_DUMPS was
set, but coreFilesPrevented was true.

Modified:
    llvm/trunk/lib/Support/Process.cpp

Modified: llvm/trunk/lib/Support/Process.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Support/Process.cpp?rev=340561&r1=340560&r2=340561&view=diff
==============================================================================
--- llvm/trunk/lib/Support/Process.cpp (original)
+++ llvm/trunk/lib/Support/Process.cpp Thu Aug 23 11:20:13 2018
@@ -87,7 +87,7 @@ static const char colorcodes[2][2][8][10
 static bool coreFilesPrevented = false;
 
 bool Process::AreCoreFilesPrevented() {
-  return !LLVM_ENABLE_CRASH_DUMPS;
+  return LLVM_ENABLE_CRASH_DUMPS ? coreFilesPrevented : false;
 }
 
 // Include the platform-specific parts of this class.