[LLVM][MIPS] Fix different definition of off_t in LLDB and LLVM

[nitesh.jain]

Summary

Fix is to define _LARGEFILE_SOURCE and _FILE_OFFSET_BITS=64. This fix will cause llvm::sys::fs::file_size() to return correct object size.

[krytarowski]

Is this just GNU specific? BSD moved to 64-bit off_t on 32-bit platforms 20+ years ago.

It's perhaps no-op, but it might be noted in the commit message what platforms are supposed to be improved.

[beanz]

Can you please move this check into HandleLLVMOptions.cmake? By putting it into a module that is vended as part of LLVM's packaging then LLVM subprojects can have consistent settings when building out-of-tree.

This would enable @zturner's request to remove the duplicated code from LLDB.

[nitesh.jain]

Update diff as per suggestion.

[nitesh.jain]

In D32125#728166, @krytarowski wrote:

Is this just GNU specific? BSD moved to 64-bit off_t on 32-bit platforms 20+ years ago.

It's perhaps no-op, but it might be noted in the commit message what platforms are supposed to be improved.

Yes.

[beanz]

This looks good to me.

[emaste]

In D32125#728166, @krytarowski wrote:

Is this just GNU specific? BSD moved to 64-bit off_t on 32-bit platforms 20+ years ago.

It's perhaps no-op, but it might be noted in the commit message what platforms are supposed to be improved.

I'd say it's better to put a comment in the source where it's set.

[nitesh.jain]

In D32125#734500, @emaste wrote:

In D32125#728166, @krytarowski wrote:

Is this just GNU specific? BSD moved to 64-bit off_t on 32-bit platforms 20+ years ago.

It's perhaps no-op, but it might be noted in the commit message what platforms are supposed to be improved.

I'd say it's better to put a comment in the source where it's set.

Added comment in the source.

[sdardis]

Hi Nitesh,

this commit broke clang-cmake-mips. Can you investigate?

http://lab.llvm.org:8011/builders/clang-cmake-mips/builds/3189

Thanks,
Simon

[nitesh.jain]

In D32125#736543, @sdardis wrote:

Hi Nitesh,

this commit broke clang-cmake-mips. Can you investigate?

http://lab.llvm.org:8011/builders/clang-cmake-mips/builds/3189

Thanks,
Simon

Hi Simon,

The assertion has been fixed in revision rL301307.

Thanks
Nitesh

CMakeLists.txt

 
 # Define an option controlling whether we should build for 32-bit on 64-bit
 # platforms, where supported.
 if( CMAKE_SIZEOF_VOID_P EQUAL 8 AND NOT WIN32 )
   # TODO: support other platforms and toolchains.
   option(LLVM_BUILD_32_BITS "Build 32 bits executables and libraries." OFF)
 endif()
 
+# If building on a 32-bit system, make sure off_t can store offsets > 2GB
+if( CMAKE_SIZEOF_VOID_P EQUAL 4 )
+  add_definitions( -D_LARGEFILE_SOURCE )
+  add_definitions( -D_FILE_OFFSET_BITS=64 )
+endif()
+
 # Define the default arguments to use with 'lit', and an option for the user to
 # override.
 set(LIT_ARGS_DEFAULT "-sv")
 if (MSVC_IDE OR XCODE)
   set(LIT_ARGS_DEFAULT "${LIT_ARGS_DEFAULT} --no-progress-bar")
 endif()
 set(LLVM_LIT_ARGS "${LIT_ARGS_DEFAULT}" CACHE STRING "Default options for lit")