Gabe Black has uploaded this change for review. (
https://gem5-review.googlesource.com/c/public/gem5/+/38195 )
Change subject: x86: Let the pseudoInst dispatch function handle the return
value.
......................................................................
x86: Let the pseudoInst dispatch function handle the return value.
When the result is returned to the caller from the pseudoInst dispatch
function, the default behavior is to not store that value using the
guestABI mechanism. In the x86 definition, I accidentally used this
version but then didn't store the result manually. The fix should simply
be to not return the result to the instruction definition and to let the
guestABI mechanism handle everything normally.
Change-Id: Ib69f266ad6314032622e5d8d69e9ff114c62657a
---
M src/arch/x86/isa/decoder/two_byte_opcodes.isa
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/arch/x86/isa/decoder/two_byte_opcodes.isa
b/src/arch/x86/isa/decoder/two_byte_opcodes.isa
index 2236c4f..f70e7bd 100644
--- a/src/arch/x86/isa/decoder/two_byte_opcodes.isa
+++ b/src/arch/x86/isa/decoder/two_byte_opcodes.isa
@@ -145,9 +145,8 @@
// instructions.
//0x04: loadall_or_reset_or_hang();
0x4: BasicOperate::gem5Op({{
- uint64_t ret;
bool recognized = PseudoInst::pseudoInst<X86PseudoInstABI>(
- xc->tcBase(), IMMEDIATE, ret);
+ xc->tcBase(), IMMEDIATE);
if (!recognized)
fault = std::make_shared<InvalidOpcode>();
}}, IsNonSpeculative);
--
To view, visit https://gem5-review.googlesource.com/c/public/gem5/+/38195
To unsubscribe, or for help writing mail filters, visit
https://gem5-review.googlesource.com/settings
Gerrit-Project: public/gem5
Gerrit-Branch: develop
Gerrit-Change-Id: Ib69f266ad6314032622e5d8d69e9ff114c62657a
Gerrit-Change-Number: 38195
Gerrit-PatchSet: 1
Gerrit-Owner: Gabe Black <gabe.bl...@gmail.com>
Gerrit-MessageType: newchange
_______________________________________________
gem5-dev mailing list -- gem5-dev@gem5.org
To unsubscribe send an email to gem5-dev-le...@gem5.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s
