Skip to content

[Arcilator] Support sim.proc.print with stream#10778

Open
nanjo712 wants to merge 2 commits into
llvm:mainfrom
nanjo712:feat/arcilator-print-to-stream
Open

[Arcilator] Support sim.proc.print with stream#10778
nanjo712 wants to merge 2 commits into
llvm:mainfrom
nanjo712:feat/arcilator-print-to-stream

Conversation

@nanjo712

@nanjo712 nanjo712 commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Handling sim.proc.print with the stream parameter in Arcilator.
Currently, only stdout and stderr are supported.

Assisted-by: Codex:GPT-5.5

@nanjo712 nanjo712 requested review from fabianschuiki, fzi-hielscher and maerhart and removed request for fabianschuiki and maerhart July 6, 2026 07:11
@circt-bot

circt-bot Bot commented Jul 6, 2026

Copy link
Copy Markdown

Results of circt-tests run for 390d8ad compared to results for f1a2831:

sv-tests

Changes in emitted diagnostics:

  • 0 total change
  • -1 error: unsupported expression: element select into hwpf_stride_pkg::hwpf_stride_status_t
  • +1 error: range of selection [31:34] from 'logic[31:0]' is reversed [-Wrange-select-reversed]

Comment thread include/circt/Dialect/Arc/Runtime/IRInterface.h
Comment thread lib/Conversion/ArcToLLVM/LowerArcToLLVM.cpp Outdated
Comment thread lib/Dialect/Arc/Runtime/ArcRuntime.cpp Outdated
Comment thread test/arcilator/sim-output-streams.mlir Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd prefer an end-to-end test added to integration_test/arcilator/JIT/sim-print.mlir

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done. Thanks!

@nanjo712

Copy link
Copy Markdown
Contributor Author

Sorry for the late reply. I couldn't process the PR in time due to a power outage at work.

@nanjo712 nanjo712 requested a review from fzi-hielscher July 10, 2026 05:40
@circt-bot

circt-bot Bot commented Jul 10, 2026

Copy link
Copy Markdown

Results of circt-tests run for 3af4e3a compared to results for f1a2831:

sv-tests

Changes in emitted diagnostics:

  • +2 total change
  • -1 error: unsupported expression: element select into hwpf_stride_pkg::hwpf_stride_status_t
  • +1 error: invalid operands to binary expression ('string' and 'bit[15:0]')
  • +1 error: invalid operands to binary expression ('string' and 'bit[7:0]')
  • +1 error: range of selection [31:34] from 'logic[31:0]' is reversed [-Wrange-select-reversed]

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.

2 participants