Skip to content

[ImportVerilog] Skip bodies for pure virtual non-void methods#10709

Closed
AmurG wants to merge 1 commit into
llvm:mainfrom
AmurG:importverilog-pure-virtual-nonvoid
Closed

[ImportVerilog] Skip bodies for pure virtual non-void methods#10709
AmurG wants to merge 1 commit into
llvm:mainfrom
AmurG:importverilog-pure-virtual-nonvoid

Conversation

@AmurG

@AmurG AmurG commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

AI generated content follows. Some of this work is gated on PRs that are yet to merge but are also filed by me before. Aware there are a lot of open PRs from me right now, but parallelizing — apologies in advance!

Skip body generation for pure virtual non-void methods in ImportVerilog (IEEE 1800-2023 §8.3.5). Previously generated empty bodies; now omits bodies entirely for pure virtuals. Maps to existing structure lowering. Standalone.

Tests: pure-virtual-nonvoid.sv.

@AmurG
AmurG requested a review from teqdruid as a code owner June 22, 2026 19:27
Consolidates A11 branches: agent11/pure-virtual-nonvoid-methods
@AmurG
AmurG force-pushed the importverilog-pure-virtual-nonvoid branch from bb1dc8a to 10b4ee7 Compare June 22, 2026 19:32
@circt-bot

circt-bot Bot commented Jun 22, 2026

Copy link
Copy Markdown

Results of circt-tests run for 10b4ee7 compared to results for 22017ad: no change to test results.

@AmurG

AmurG commented Jul 8, 2026

Copy link
Copy Markdown
Contributor Author

Closing (same reset as #10697). This returns beside the class-lowering changes that motivate it.

@AmurG AmurG closed this Jul 8, 2026
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.

1 participant