[PATCH] D52023: [ThinLTO]Allow setting of maximum cache size with 64-bit number, and provide C-interface function for large values

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 13 04:03:51 PDT 2018


jhenderson created this revision.
jhenderson added reviewers: mehdi_amini, tejohnson, pcc, kromanova, steven_wu, deadalnix.
Herald added subscribers: dexonsmith, eraman, inglorion.

The maximum cache size in terms of bytes is a 64-bit number. However, the methods to set it only take `unsigned`, which means that the maximum cache size cannot be specified above 4GB. That's quite small compared to the output of some projects, so it makes sense to provide the ability to set larger values in that field.

We also need a C-interface function that provides a greater range than the existing `thinlto_codegen_set_cache_size_bytes`, which also only takes an `unsigned`, so this change also adds `thinlto_codegen_set_cache_size_megabytes`.


Repository:
  rL LLVM

https://reviews.llvm.org/D52023

Files:
  include/llvm-c/lto.h
  include/llvm/LTO/legacy/ThinLTOCodeGenerator.h
  test/ThinLTO/X86/cache.ll
  tools/llvm-lto/llvm-lto.cpp
  tools/lto/lto.cpp
  tools/lto/lto.exports

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52023.165245.patch
Type: text/x-patch
Size: 4260 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180913/8c496abe/attachment-0001.bin>


More information about the llvm-commits mailing list