Skip to content

test i386#25520

Closed
ringabout wants to merge 13 commits into
develfrom
pr_Kaste
Closed

test i386#25520
ringabout wants to merge 13 commits into
develfrom
pr_Kaste

Conversation

@ringabout

Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings February 16, 2026 08:28
@ringabout ringabout marked this pull request as draft February 16, 2026 08:32

This comment was marked as outdated.

@ringabout

Copy link
Copy Markdown
Member Author

@copilot Make it work on i386 platform, i.e., the CI should be green. Especially, nim c -r --lib:lib --mm:refc -d:useGcAssert -d:useSysAssert tests/align/talign.nim need to work.

Copilot AI commented Feb 16, 2026

Copy link
Copy Markdown
Contributor

@ringabout I've opened a new pull request, #25521, to work on those changes. Once the pull request is ready, I'll request review from you.

@ringabout

Copy link
Copy Markdown
Member Author

@copilot Make it work on i386 platform, i.e., the CI should be green. Especially, nim c -r --lib:lib --mm:refc -d:useGcAssert -d:useSysAssert tests/align/talign.nim need to work. Notes failures are caused by #25476

Copilot AI commented Feb 16, 2026

Copy link
Copy Markdown
Contributor

@ringabout I've opened a new pull request, #25522, to work on those changes. Once the pull request is ready, I'll request review from you.

@ringabout

Copy link
Copy Markdown
Member Author

@copilot Make it work on i386 platform, i.e., the CI should be green. Especially, nim c -r --lib:lib --mm:refc -d:useGcAssert -d:useSysAssert tests/align/talign.nim need to work. Notes failures are caused by #25476

Copilot AI commented Feb 16, 2026

Copy link
Copy Markdown
Contributor

@ringabout I've opened a new pull request, #25523, to work on those changes. Once the pull request is ready, I'll request review from you.

@ringabout

Copy link
Copy Markdown
Member Author

@copilot Make it work on i386 platform, i.e., the CI should be green. Especially, nim c -r --lib:lib --mm:refc -d:useGcAssert -d:useSysAssert tests/align/talign.nim need to work. Notes failures are caused by #25476

Copilot AI commented Feb 16, 2026

Copy link
Copy Markdown
Contributor

@ringabout I've opened a new pull request, #25524, to work on those changes. Once the pull request is ready, I'll request review from you.

Simplified alignment checks by removing conditional statements.
@ringabout ringabout closed this Feb 24, 2026
@ringabout ringabout deleted the pr_Kaste branch February 24, 2026 08:44
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.

3 participants