Skip to content

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

Draft
kfcripps wants to merge 8 commits into
mainfrom
4298-upstream
Draft

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

Conversation

@kfcripps

Copy link
Copy Markdown
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
Comment thread frontends/common/resolveReferences/resolveReferences.cpp
Comment thread frontends/p4/actionsInlining.h Outdated
Comment thread frontends/p4/frontend.cpp Outdated
@kfcripps

kfcripps commented Feb 5, 2024

Copy link
Copy Markdown
Owner Author

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