swiftlang/swift
[SR-6264] Unify duplicated code in GlobalPropertyOpt::isVisibleExternally() and FunctionLivenessComputation::isVisibleExternally()
オープン
#48,814 opened on 2017/11/01
IRGencompilergood first issueimprovement
Repository metrics
- Stars
- (69,989 個のスター)
- PR merge metrics
- (平均マージ 8d 17h) (30d で 510 merged PRs)
説明
| Previous ID | SR-6264 |
| Radar | None |
| Original Reporter | @gparker42 |
| Type | Improvement |
| Votes | 0 |
| Component/s | Compiler |
| Labels | Improvement, IRGen, StarterBug |
| Assignee | None |
| Priority | Medium |
md5: 7fe2948a0ff34a9d22f375125e2fe476
Issue Description:
Functions GlobalPropertyOpt::isVisibleExternally() and FunctionLivenessComputation::isVisibleExternally() have duplicated code that checks whether a declaration may be externally used based on its access level. This duplication should be fixed.
Possible solution: add a function to class ValueDecl to do the work, and call it from both places.