[PR #202] [CLOSED] tests: fix jcc when gas using same register for %0 and %1 #229

Open
opened 2026-01-31 21:30:21 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/qemu/qemu/pull/202
Author: @xor2003
Created: 12/29/2021
Status: Closed

Base: masterHead: master


📝 Commits (1)

  • 0ffb961 tests: fix jcc when gas using same register for %0 and %1

📊 Changes

1 file changed (+4 additions, -4 deletions)

View changed files

📝 tests/tcg/i386/test-i386.c (+4 -4)

📄 Description

The JCC test happening is not as expected to be:
For example v1 is in eax, and v2 is in edx.
But res is also in eax.
And it is being overwritten just before comparison.
Or for setCC the result is set to lower byte but ew bit register is being garbaged with v1 value.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/qemu/qemu/pull/202 **Author:** [@xor2003](https://github.com/xor2003) **Created:** 12/29/2021 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`0ffb961`](https://github.com/qemu/qemu/commit/0ffb96126b87c955b7805fd37a7ddf371b366ca0) tests: fix jcc when gas using same register for %0 and %1 ### 📊 Changes **1 file changed** (+4 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `tests/tcg/i386/test-i386.c` (+4 -4) </details> ### 📄 Description The JCC test happening is not as expected to be: For example v1 is in eax, and v2 is in edx. But res is also in eax. And it is being overwritten just before comparison. Or for setCC the result is set to lower byte but ew bit register is being garbaged with v1 value. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 21:30:21 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/qemu#229