Skip to content

Go: make parameter nodes for unused parameters #3 (Base ParameterNode on IR::InitParameterInstruction instead of SsaNode)#13673

Closed
owen-mc wants to merge 3 commits intogithub:mainfrom
owen-mc:go/make-parameter-nodes-for-unused-parameters-3
Closed

Go: make parameter nodes for unused parameters #3 (Base ParameterNode on IR::InitParameterInstruction instead of SsaNode)#13673
owen-mc wants to merge 3 commits intogithub:mainfrom
owen-mc:go/make-parameter-nodes-for-unused-parameters-3

Conversation

@owen-mc
Copy link
Contributor

@owen-mc owen-mc commented Jul 5, 2023

An alternative approach to #13664

@owen-mc
Copy link
Contributor Author

owen-mc commented Jul 12, 2023

Closed in favour of #13672. As it stands this approach stops data from flowing through ParameterNodes. It is probably possible to fix that, but the more superficial approach of #13672 avoids it entirely.

@owen-mc owen-mc closed this Jul 12, 2023
@owen-mc owen-mc deleted the go/make-parameter-nodes-for-unused-parameters-3 branch July 18, 2023 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant