[PATCH] D92900: [ThinLTO] Add Visibility bits to GlobalValueSummary::GVFlags

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 17:36:41 PST 2020


MaskRay created this revision.
MaskRay added reviewers: pcc, tejohnson.
Herald added subscribers: dexonsmith, arphaman, steven_wu, hiraditya, inglorion.
MaskRay requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Imported functions and variables get the visibility from the prevailing module.
However, non-prevailing definitions do not get the visibility.

This patch adds visibility bits to GlobalValueSummary::GVFlags so that
non-prevailing definitions can get the constraining visibility from the
prevailing definitions.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D92900

Files:
  llvm/include/llvm/IR/ModuleSummaryIndex.h
  llvm/include/llvm/IR/ModuleSummaryIndexYAML.h
  llvm/lib/Analysis/ModuleSummaryAnalysis.cpp
  llvm/lib/AsmParser/LLLexer.cpp
  llvm/lib/AsmParser/LLParser.cpp
  llvm/lib/AsmParser/LLToken.h
  llvm/lib/Bitcode/Reader/BitcodeReader.cpp
  llvm/lib/Bitcode/Writer/BitcodeWriter.cpp
  llvm/lib/IR/AsmWriter.cpp
  llvm/lib/IR/ModuleSummaryIndex.cpp
  llvm/lib/LTO/LTO.cpp
  llvm/lib/Transforms/IPO/FunctionImport.cpp
  llvm/test/Assembler/multi-summary-disassemble.ll
  llvm/test/Assembler/thinlto-multiple-summaries-for-guid.ll
  llvm/test/Assembler/thinlto-summary-visibility.ll
  llvm/test/Assembler/thinlto-summary.ll
  llvm/test/Assembler/thinlto-vtable-summary.ll
  llvm/test/Bitcode/thinlto-alias.ll
  llvm/test/Bitcode/thinlto-function-summary-callgraph-profile-summary.ll
  llvm/test/Bitcode/thinlto-function-summary-callgraph-relbf.ll
  llvm/test/Bitcode/thinlto-function-summary-refgraph.ll
  llvm/test/Bitcode/thinlto-type-tests.ll
  llvm/test/Bitcode/thinlto-type-vcalls.ll
  llvm/test/ThinLTO/X86/funcimport_alwaysinline.ll
  llvm/test/ThinLTO/X86/load-store-caching.ll
  llvm/test/ThinLTO/X86/visibility.ll
  llvm/test/Transforms/LowerTypeTests/import-unsat.ll
  llvm/test/Transforms/WholeProgramDevirt/import-indir.ll
  llvm/test/tools/gold/X86/thinlto.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D92900.310392.patch
Type: text/x-patch
Size: 74784 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201209/d7982359/attachment.bin>


More information about the llvm-commits mailing list