253436 2010-10-06 23:26:23 +0000 vex amd64->IR: unhandled instruction bytes: 0xF2 0xA6 (repne cmps) 2026-01-24 10:01:57 +0000 1 1 6 Developer tools valgrind vex 3.6 SVN Unlisted Binaries Linux RESOLVED FIXED NOR normal --- 253451 0 vince pjfloyd itislu.git jacobly.alt pjfloyd researchlover44 0 oldest_to_newest 1028613 0 vince 2010-10-06 23:26:23 +0000 Version: 3.6 SVN OS: Linux The "repne cmps" string instruction is not supported on amd64 Reproducible: Didn't try 1901614 1 125021 jacobly.alt 2020-01-10 17:53:37 +0000 Created attachment 125021 Added missing cmps variants to VEX amd64. I was trying to run valgrind on some assembly code and it also chokes on a plain old cmps. I implemented both in the attached patch. 2079584 2 pjfloyd 2021-11-22 11:12:12 +0000 Do you have a testcase for this? 2486476 3 researchlover44 2026-01-19 21:48:22 +0000 (In reply to Paul Floyd from comment #2) > Do you have a testcase for this? Hi! Can you point me to where I could provide this testcase? BR 2486592 4 pjfloyd 2026-01-20 09:37:47 +0000 Ideally just attach a small testcase here. 2486780 5 188721 itislu.git 2026-01-20 19:07:44 +0000 Created attachment 188721 Uncomment tests for cmps variants for VEX amd64. The test cases already existed, they were just commented out. When applying this patch, valgrind reports illegal instructions when running the compiled test-amd64.c. With the patch from attachment 125021 applied, it does not report any issues anymore. 2487245 6 pjfloyd 2026-01-22 06:03:11 +0000 ! I should have done a git rep before asking. 2487960 7 pjfloyd 2026-01-24 10:01:57 +0000 Thanks for the contributions! commit 1f5118fffdfbfb40405b8493ad6e8fee343c5759 (HEAD -> master, origin/users/paulf/try-bug253436, origin/master, origin/HEAD, bug253436) Author: Paul Floyd <pjfloyd@wanadoo.fr> Date: Sat Jan 24 08:20:29 2026 +0100 Bug 253436 - vex amd64->IR: unhandled instruction bytes: 0xF2 0xA6 (repne cmps) Patch submitted by jacobly.alt@gmail.com Regtest update submitted by Lukas Dullinger <itislu.git@gmail.com> 125021 2020-01-10 17:53:37 +0000 2020-01-10 17:53:37 +0000 Added missing cmps variants to VEX amd64. cmps.patch text/plain 1849 jacobly.alt ZGlmZiAtLWdpdCBhL1ZFWC9wcml2L2d1ZXN0X2FtZDY0X3RvSVIuYyBiL1ZFWC9wcml2L2d1ZXN0 X2FtZDY0X3RvSVIuYwppbmRleCBmYWRmNDdkNDEuLjg5OGQzZDdlYiAxMDA2NDQKLS0tIGEvVkVY L3ByaXYvZ3Vlc3RfYW1kNjRfdG9JUi5jCisrKyBiL1ZFWC9wcml2L2d1ZXN0X2FtZDY0X3RvSVIu YwpAQCAtMjA3MDEsOCArMjA3MDEsOCBAQCBMb25nIGRpc19FU0NfTk9ORSAoCiAgICAgICAgICBk aXNfUkVQX29wICggZHJlcywgQU1ENjRDb25kQWx3YXlzLCBkaXNfTU9WUywgc3osCiAgICAgICAg ICAgICAgICAgICAgICAgZ3Vlc3RfUklQX2N1cnJfaW5zdHIsCiAgICAgICAgICAgICAgICAgICAg ICAgZ3Vlc3RfUklQX2Jic3RhcnQrZGVsdGEsICJyZXAgbW92cyIsIHBmeCApOwotICAgICAgICBk cmVzLT53aGF0TmV4dCA9IERpc19TdG9wSGVyZTsKLSAgICAgICAgcmV0dXJuIGRlbHRhOworICAg ICAgICAgdmFzc2VydChkcmVzLT53aGF0TmV4dCA9PSBEaXNfU3RvcEhlcmUpOworICAgICAgICAg cmV0dXJuIGRlbHRhOwogICAgICAgfQogICAgICAgLyogQTQ6IG1vdnNiICovCiAgICAgICBpZiAo IWhhdmVGMyhwZngpICYmICFoYXZlRjIocGZ4KSkgewpAQCAtMjA3MTUsMTQgKzIwNzE1LDMxIEBA IExvbmcgZGlzX0VTQ19OT05FICgKIAogICAgY2FzZSAweEE2OgogICAgY2FzZSAweEE3OgotICAg ICAgLyogRjMgQTYvQTc6IHJlcGUgY21wcy9yZXAgY21wc3t3LGwscX0gKi8KLSAgICAgIGlmICho YXZlRjMocGZ4KSAmJiAhaGF2ZUYyKHBmeCkpIHsKKyAgICAgIC8qIEYyIEE2L0E3OiByZXBuZSBj bXBzYi9yZXBuZSBjbXBze3csbCxxfSAqLworICAgICAgaWYgKGhhdmVGMihwZngpICYmICFoYXZl RjMocGZ4KSkgeworICAgICAgICAgaWYgKG9wYyA9PSAweEE2KQorICAgICAgICAgICAgc3ogPSAx OworICAgICAgICAgZGlzX1JFUF9vcCAoIGRyZXMsIEFNRDY0Q29uZE5aLCBkaXNfQ01QUywgc3os IAorICAgICAgICAgICAgICAgICAgICAgIGd1ZXN0X1JJUF9jdXJyX2luc3RyLAorICAgICAgICAg ICAgICAgICAgICAgIGd1ZXN0X1JJUF9iYnN0YXJ0K2RlbHRhLCAicmVwbmUgY21wcyIsIHBmeCAp OworICAgICAgICAgdmFzc2VydChkcmVzLT53aGF0TmV4dCA9PSBEaXNfU3RvcEhlcmUpOworICAg ICAgICAgcmV0dXJuIGRlbHRhOworICAgICAgfQorICAgICAgLyogRjMgQTYvQTc6IHJlcGUgY21w c2IvcmVwZSBjbXBze3csbCxxfSAqLworICAgICAgaWYgKCFoYXZlRjIocGZ4KSAmJiBoYXZlRjMo cGZ4KSkgewogICAgICAgICAgaWYgKG9wYyA9PSAweEE2KQogICAgICAgICAgICAgc3ogPSAxOwog ICAgICAgICAgZGlzX1JFUF9vcCAoIGRyZXMsIEFNRDY0Q29uZFosIGRpc19DTVBTLCBzeiwgCiAg ICAgICAgICAgICAgICAgICAgICAgZ3Vlc3RfUklQX2N1cnJfaW5zdHIsCiAgICAgICAgICAgICAg ICAgICAgICAgZ3Vlc3RfUklQX2Jic3RhcnQrZGVsdGEsICJyZXBlIGNtcHMiLCBwZnggKTsKLSAg ICAgICAgIGRyZXMtPndoYXROZXh0ID0gRGlzX1N0b3BIZXJlOworICAgICAgICAgdmFzc2VydChk cmVzLT53aGF0TmV4dCA9PSBEaXNfU3RvcEhlcmUpOworICAgICAgICAgcmV0dXJuIGRlbHRhOwor ICAgICAgfQorICAgICAgLyogQTYvQTc6IGNtcHNiL2NtcHN7dyxsLHF9ICovCisgICAgICBpZiAo IWhhdmVGMihwZngpICYmICFoYXZlRjMocGZ4KSkgeworICAgICAgICAgaWYgKG9wYyA9PSAweEE2 KQorICAgICAgICAgICAgc3ogPSAxOworICAgICAgICAgZGlzX3N0cmluZ19vcCAoIGRpc19DTVBT LCBzeiwgImNtcHMiLCBwZnggKTsKICAgICAgICAgIHJldHVybiBkZWx0YTsKICAgICAgIH0KICAg ICAgIGdvdG8gZGVjb2RlX2ZhaWx1cmU7Cg== 188721 2026-01-20 19:07:44 +0000 2026-01-20 19:07:44 +0000 Uncomment tests for cmps variants for VEX amd64. test-amd64.patch text/plain 502 itislu.git ZGlmZiAtLWdpdCBhL1ZFWC90ZXN0L3Rlc3QtYW1kNjQuYyBiL1ZFWC90ZXN0L3Rlc3QtYW1kNjQu YwppbmRleCBiZWRjOTllOTguLjk4MWFhM2VhMCAxMDA2NDQKLS0tIGEvVkVYL3Rlc3QvdGVzdC1h bWQ2NC5jCisrKyBiL1ZFWC90ZXN0L3Rlc3QtYW1kNjQuYwpAQCAtMTIwNyw5ICsxMjA3LDkgQEAg dm9pZCB0ZXN0X3N0cmluZyh2b2lkKQogICAgVEVTVF9TVFJJTkcoc2NhcywgIiIpOwogICAgLy8g UkVJTlNUQVRFOiBURVNUX1NUUklORyhzY2FzLCAicmVweiAiKTsKICAgIFRFU1RfU1RSSU5HKHNj YXMsICJyZXBueiAiKTsKLSAgIC8vIFJFSU5TVEFURTogVEVTVF9TVFJJTkcoY21wcywgIiIpOwor ICAgVEVTVF9TVFJJTkcoY21wcywgIiIpOwogICAgVEVTVF9TVFJJTkcoY21wcywgInJlcHogIik7 Ci0gICAvLyBSRUlOU1RBVEU6IFRFU1RfU1RSSU5HKGNtcHMsICJyZXBueiAiKTsKKyAgIFRFU1Rf U1RSSU5HKGNtcHMsICJyZXBueiAiKTsKIH0KIAogLyogVk04NiB0ZXN0ICovCg==