[code-reflection] Integrated: Remove Op::traverseOperands

Paul Sandoz psandoz at openjdk.org
Wed Jul 9 19:38:58 UTC 2025


On Wed, 9 Jul 2025 17:43:30 GMT, Paul Sandoz <psandoz at openjdk.org> wrote:

> Remove `Op::traverseOperands`. Such traversal methods tend to be limited because it is not possible to control the traversal, and adding such facility will make the API more complex, likely more so that just writing the specific traversal method, since its only a few lines of code.
> 
> Evidence in other areas of the code base that traverses up the tree of values indicates this is so, since this method is not used. A better approach would to broaden the functionality of `Value::dependsOn`, adding another method that returns a lazy stream of all dependent values, in order.

This pull request has now been integrated.

Changeset: 1935a8e1
Author:    Paul Sandoz <psandoz at openjdk.org>
URL:       https://git.openjdk.org/babylon/commit/1935a8e15f6d01e282f51ecccb6b477590ba5d45
Stats:     31 lines in 3 files changed: 6 ins; 24 del; 1 mod

Remove Op::traverseOperands

-------------

PR: https://git.openjdk.org/babylon/pull/490


More information about the babylon-dev mailing list