Skip to content

Do not check for shadowing repeatedly in inlining passes.#3

Draft
kfcripps wants to merge 8 commits intomainfrom
4298-upstream
Draft

Do not check for shadowing repeatedly in inlining passes.#3
kfcripps wants to merge 8 commits intomainfrom
4298-upstream

Conversation

@kfcripps
Copy link
Owner

No description provided.

@kfcripps kfcripps changed the title Check for shadowing after all inlining passes have completed. Do not check for shadowing repeatedly in inlining passes. Jan 31, 2024
@kfcripps
Copy link
Owner Author

kfcripps commented Feb 5, 2024

Applied to A P4:

real    1m54.165s
user    1m45.484s
sys     0m9.316s

Applied to A P4+:

real    16m36.850s
user    16m24.559s
sys     0m15.304s

fruffy and others added 5 commits February 7, 2024 21:42
… end. Move hex separator utility to common class. (p4lang#4400)
Implement copy constructor for hvec_map. We cannot use a default one since it would involved copying of std::pair<const KEY, VAL> that had its copy ctor explicitly disabled.

Instead, go the standard "way" for maps via clear() and reinsert().

Fixes p4lang#4331
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants