Skip to content

Commit 07f8b35

Browse files
committed
Data flow: Update documentation
1 parent db8b6ac commit 07f8b35

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

docs/ql-libraries/dataflow/dataflow.md

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -294,7 +294,10 @@ through an additional step targeting a `PostUpdateNode`).
294294

295295
It is recommended to introduce `PostUpdateNode`s for all `ArgumentNode`s (this
296296
can be skipped for immutable arguments), and all field qualifiers for both
297-
reads and stores.
297+
reads and stores. Note also that in the case of compund arguments, such as
298+
`b ? x : y`, it is recommented to have post-update nodes for `x` and `y` (and
299+
not the compound argument itself), and let `[post update] x` have both `x`
300+
and `b ? x : y` as pre-update nodes (and similarly for `[post update] y`).
298301

299302
Remember to define local flow for `PostUpdateNode`s as well in
300303
`simpleLocalFlowStep`. In general out-going local flow from `PostUpdateNode`s

0 commit comments

Comments
 (0)