diff options
| author | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2014-05-07 23:00:22 +0000 |
|---|---|---|
| committer | Duncan P. N. Exon Smith <dexonsmith@apple.com> | 2014-05-07 23:00:22 +0000 |
| commit | e60adfdbd0d8ee91d6b48d29381d68acf192b385 (patch) | |
| tree | d3266b0faa5c8fde2b39dbdd53a22a4aee011a4f /llvm/lib/Transforms/IPO/ExtractGV.cpp | |
| parent | b80de1012ae346786dbb019ed6bdf8ba8820d8ab (diff) | |
| download | bcm5719-llvm-e60adfdbd0d8ee91d6b48d29381d68acf192b385.tar.gz bcm5719-llvm-e60adfdbd0d8ee91d6b48d29381d68acf192b385.zip | |
GlobalValue: Assert symbols with local linkage have default visibility
The change to ExtractGV.cpp has no functionality change except to avoid
the asserts. Existing testcases already cover this, so I didn't add a
new one.
llvm-svn: 208264
Diffstat (limited to 'llvm/lib/Transforms/IPO/ExtractGV.cpp')
| -rw-r--r-- | llvm/lib/Transforms/IPO/ExtractGV.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/llvm/lib/Transforms/IPO/ExtractGV.cpp b/llvm/lib/Transforms/IPO/ExtractGV.cpp index 408713a2c24..40ec9fa8c1d 100644 --- a/llvm/lib/Transforms/IPO/ExtractGV.cpp +++ b/llvm/lib/Transforms/IPO/ExtractGV.cpp @@ -27,11 +27,10 @@ using namespace llvm; /// the split module remain valid. static void makeVisible(GlobalValue &GV, bool Delete) { bool Local = GV.hasLocalLinkage(); - if (Local) - GV.setVisibility(GlobalValue::HiddenVisibility); - if (Local || Delete) { GV.setLinkage(GlobalValue::ExternalLinkage); + if (Local) + GV.setVisibility(GlobalValue::HiddenVisibility); return; } |
