Skip to content

Commit b3189f8

Browse files
committed
C#: Replace Ssa::UncertainDefinition with SsaUncertainWrite.
1 parent a683c15 commit b3189f8

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

csharp/ql/lib/semmle/code/csharp/dataflow/SSA.qll

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -368,7 +368,7 @@ module Ssa {
368368
*/
369369
private Definition getAPhiInputOrPriorDefinition() {
370370
result = this.(SsaPhiDefinition).getAnInput() or
371-
result = this.(UncertainDefinition).getPriorDefinition()
371+
result = this.(SsaUncertainWrite).getPriorDefinition()
372372
}
373373

374374
/**
@@ -691,12 +691,14 @@ module Ssa {
691691
}
692692

693693
/**
694+
* DEPRECATED: Use `SsaUncertainWrite` instead.
695+
*
694696
* An SSA definition that represents an uncertain update of the underlying
695697
* assignable. Either an explicit update that is uncertain (`ref` assignments
696698
* need not be certain), an implicit non-local update via a call, or an
697699
* uncertain update of the qualifier.
698700
*/
699-
class UncertainDefinition extends Definition, SsaImpl::UncertainWriteDefinition {
701+
deprecated class UncertainDefinition extends Definition, SsaImpl::UncertainWriteDefinition {
700702
/**
701703
* Gets the immediately preceding definition. Since this update is uncertain,
702704
* the value from the preceding definition might still be valid.

csharp/ql/lib/semmle/code/csharp/dataflow/internal/SsaImpl.qll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -897,7 +897,7 @@ private module Cached {
897897
}
898898

899899
cached
900-
Definition uncertainWriteDefinitionInput(UncertainWriteDefinition def) {
900+
deprecated Definition uncertainWriteDefinitionInput(UncertainWriteDefinition def) {
901901
Impl::uncertainWriteDefinitionInput(def, result)
902902
}
903903

0 commit comments

Comments
 (0)