[Patch] x86 implementation of RegModifiedByInstruction
Original Reporter info from Mantis: CuriousKit @CuriousKit
-
Reporter name: J. Gareth Moreton
Original Reporter info from Mantis: CuriousKit @CuriousKit
- Reporter name: J. Gareth Moreton
Description:
Currently, the default implementation of RegModifiedByInstruction is not fail-safe (it returns False even if an instruction modifies a register) and this default implementation is used for a new MOV optimisation (I didn't program this). In the supplied patch is a fairly exhaustive implementation of the RegModifiedByInstruction for i386 and x86_64.
Steps to reproduce:
Apply patch and confirm correct compilation of compiler and test projects under -O3.
Additional information:
An internal error was added to RegUsedByInstruction to suppress a potential warning from a case block not covering all eventualities.
Mantis conversion info:
- Mantis ID: 36376
- OS: Microsoft Windows
- OS Build: 10 Professional
- Build: r43600
- Platform: i386 and x86_64
- Version: 3.3.1
- Fixed in version: 3.3.1
- Fixed in revision: 43611 (#4525edd9)