[PATCH] D70404: [ThinLTO] Always import constants

Eugene Leviant via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 21 07:03:40 PST 2019


evgeny777 updated this revision to Diff 230454.
evgeny777 added a comment.

I reimplemented the patch, now using extra 'Constant' attribute in GlobalVarSummary.

The goal is to allow importing of constants, which are not being detected as 'ReadOnly'. Such constants are imported with available_externally linkage, still allowing compiler to make some optimizations along the way.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70404/new/

https://reviews.llvm.org/D70404

Files:
  llvm/include/llvm/IR/ModuleSummaryIndex.h
  llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
  llvm/lib/AsmParser/LLParser.cpp
  llvm/lib/Bitcode/Reader/BitcodeReader.cpp
  llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
  llvm/lib/IR/AsmWriter.cpp
  llvm/lib/IR/ModuleSummaryIndex.cpp
  llvm/test/Assembler/thinlto-summary.ll
  llvm/test/Assembler/thinlto-vtable-summary.ll
  llvm/test/Bitcode/thinlto-function-summary-refgraph.ll
  llvm/test/ThinLTO/X86/Inputs/import-constant.ll
  llvm/test/ThinLTO/X86/import-constant.ll
  llvm/test/ThinLTO/X86/load-store-caching.ll
  llvm/test/ThinLTO/X86/referenced_by_constant.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D70404.230454.patch
Type: text/x-patch
Size: 21141 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191121/5b96993a/attachment.bin>


More information about the llvm-commits mailing list