Skip to content

RISC-V: provide factory methods for MemoryReference objects#7209

Closed
janvrany wants to merge 3 commits into
eclipse-omr:masterfrom
janvrany:pr/riscv-memoryreference-factories
Closed

RISC-V: provide factory methods for MemoryReference objects#7209
janvrany wants to merge 3 commits into
eclipse-omr:masterfrom
janvrany:pr/riscv-memoryreference-factories

Conversation

@janvrany

Copy link
Copy Markdown
Contributor

No description provided.

@janvrany

Copy link
Copy Markdown
Contributor Author

jenkins build riscv

@janvrany

janvrany commented Nov 8, 2025

Copy link
Copy Markdown
Contributor Author

jenkins build riscv

@janvrany janvrany force-pushed the pr/riscv-memoryreference-factories branch from 90d28e2 to b9b5f8c Compare November 8, 2025 06:48
@janvrany

janvrany commented Nov 8, 2025

Copy link
Copy Markdown
Contributor Author

jenkins build riscv

@hzongaro hzongaro left a comment

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.

Just some questions about changes that aren’t central to this pull request.

Comment thread compiler/riscv/codegen/MemoryReference.hpp Outdated
Comment thread compiler/riscv/codegen/OMRMemoryReference.hpp Outdated
Comment thread compiler/riscv/codegen/OMRTreeEvaluator.hpp Outdated
@hzongaro hzongaro self-assigned this Nov 14, 2025
@janvrany janvrany force-pushed the pr/riscv-memoryreference-factories branch from b9b5f8c to 0d1de08 Compare November 17, 2025 12:12
@janvrany

Copy link
Copy Markdown
Contributor Author

jenkins build riscv

3 similar comments
@janvrany

Copy link
Copy Markdown
Contributor Author

jenkins build riscv

@janvrany

janvrany commented Apr 1, 2026

Copy link
Copy Markdown
Contributor Author

jenkins build riscv

@janvrany

Copy link
Copy Markdown
Contributor Author

jenkins build riscv

@janvrany

janvrany commented May 8, 2026

Copy link
Copy Markdown
Contributor Author

Closing as being superseded by #8245

@janvrany janvrany closed this May 8, 2026
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.

2 participants