[Lldb-commits] [lldb] r214291 - Use __linux__ macro throughout, instead of ocasional __linux.
Sylvestre Ledru
sylvestre at debian.org
Wed Jul 30 01:33:21 PDT 2014
Author: sylvestre
Date: Wed Jul 30 03:33:21 2014
New Revision: 214291
URL: http://llvm.org/viewvc/llvm-project?rev=214291&view=rev
Log:
Use __linux__ macro throughout, instead of ocasional __linux.
__linux is not universally defined across all standards versions, compilers and architectures. Specifically at C99 and up, it's not defined in GCC on powerpc platform.
See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=28314
Bugzilla: http://llvm.org/bugs/show_bug.cgi?id=20380
Patch by Dimitri John Ledkov <dimitri.ledkov at canonical.com>
Modified:
lldb/trunk/tools/lldb-mi/MICmnLogMediumFile.cpp
lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.cpp
lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.h
Modified: lldb/trunk/tools/lldb-mi/MICmnLogMediumFile.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/lldb-mi/MICmnLogMediumFile.cpp?rev=214291&r1=214290&r2=214291&view=diff
==============================================================================
--- lldb/trunk/tools/lldb-mi/MICmnLogMediumFile.cpp (original)
+++ lldb/trunk/tools/lldb-mi/MICmnLogMediumFile.cpp Wed Jul 30 03:33:21 2014
@@ -28,7 +28,7 @@
#if defined( _MSC_VER )
#include "MIUtilSystemWindows.h"
-#elif defined( __FreeBSD__ ) || defined( __linux )
+#elif defined( __FreeBSD__ ) || defined( __linux__ )
#include "MIUtilSystemLinux.h"
#elif defined( __APPLE__ )
#include "MIUtilSystemOsx.h"
Modified: lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.cpp
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.cpp?rev=214291&r1=214290&r2=214291&view=diff
==============================================================================
--- lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.cpp (original)
+++ lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.cpp Wed Jul 30 03:33:21 2014
@@ -22,7 +22,7 @@
// Include compiler configuration
#include "MICmnConfig.h"
-#if defined( __FreeBSD__ ) || defined( __linux )
+#if defined( __FreeBSD__ ) || defined( __linux__ )
// In-house headers:
#include "MIUtilSystemLinux.h"
@@ -119,4 +119,4 @@ bool CMIUtilSystemLinux::GetLogFilesPath
return MIstatus::success;
}
-#endif // #if defined( __linux )
+#endif // #if defined( __linux__ )
Modified: lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.h
URL: http://llvm.org/viewvc/llvm-project/lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.h?rev=214291&r1=214290&r2=214291&view=diff
==============================================================================
--- lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.h (original)
+++ lldb/trunk/tools/lldb-mi/MIUtilSystemLinux.h Wed Jul 30 03:33:21 2014
@@ -24,7 +24,7 @@
// Include compiler configuration
#include "MICmnConfig.h"
-#if defined( __FreeBSD__ ) || defined( __linux )
+#if defined( __FreeBSD__ ) || defined( __linux__ )
// In-house headers:
#include "MIUtilString.h"
@@ -57,4 +57,4 @@ public:
typedef CMIUtilSystemLinux CMIUtilSystem;
-#endif // #if defined( __linux )
+#endif // #if defined( __linux__ )
More information about the lldb-commits
mailing list