Development:InstCountCI - Revision history

Neobrain: BUILD_TESTS -> BUILD_TESTING

2025-09-11T12:11:54Z

BUILD_TESTS -> BUILD_TESTING

2024-01-30T03:43:12Z

Diving deeper in to the assembly

2023-11-30T20:13:34Z

2023-11-30T20:12:57Z

2023-11-30T20:11:47Z

2023-09-30T13:20:41Z

2023-08-24T00:23:53Z

2023-08-23T08:44:41Z

2023-08-23T08:33:40Z

2023-08-23T08:28:36Z

@@ Line 11: / Line 11: @@
 Some additional cmake options need to be passed to the FEX-Emu cmake options to get the tests building.
-* -DBUILD_TESTS=True
+* -DBUILD_TESTING=True
 * -DENABLE_VIXL_DISASSEMBLER=True

@@ Line 57: / Line 57: @@
 == Diving deeper in to the assembly ==
 While the instruction count CI is good at showing the final result, it isn't the best at showing what FEX did to get to that result. This is where the assembly test harness can come in handy.
 * Create a file in '''unittests/ASM/Test.asm'''

@@ Line 6: / Line 6: @@
 === What you need ===
 * An Arm64 Linux device that can build FEX
 === Additional cmake options ===

← Older revision		Revision as of 20:12, 30 November 2023
Line 106:		Line 106:
	* Specifically the options for '''FEX_PASSMANAGERDUMPIR''' to get more IR dumping options and '''FEX_HOSTFEATURES''' to fake CPU feature support.		* Specifically the options for '''FEX_PASSMANAGERDUMPIR''' to get more IR dumping options and '''FEX_HOSTFEATURES''' to fake CPU feature support.
	* Enabling the vixl simulator with the cmake option '''-DENABLE_VIXL_SIMULATOR=True''' can be useful to test features your CPU doesn't support!		* Enabling the vixl simulator with the cmake option '''-DENABLE_VIXL_SIMULATOR=True''' can be useful to test features your CPU doesn't support!
−	** '''Be warned! This invalidates InstCountCI results since it augments x87 instruction counts which need to jump out of the JIT.

@@ Line 35: / Line 35: @@
         "movd mm0, eax": {
    -      "ExpectedInstructionCount": 3,
    +      "ExpectedInstructionCount": 2,
           "Comment": "0x0f 0x6e",
           "ExpectedArm64ASM": [

@@ Line 32: / Line 32: @@
 === Example ===
-Minor improvement when optimizing a move instruction
+Minor improvement when optimizing a move instruction. From [https://github.com/FEX-Emu/FEX/pull/2972 This] pull request.
         "movd mm0, eax": {
    -      "ExpectedInstructionCount": 3,

@@ Line 47: / Line 47: @@
         },
 * Reset the files with '''git checkout -- unittests/InstructionCountCI/*.json''' if the changes weren't desired.
 == Diving deeper in to the assembly ==

← Older revision		Revision as of 08:28, 23 August 2023
Line 30:		Line 30:

	Now to see how the implementations have changed, you can just run '''git diff''' to see how the json files in '''unittests/InstructionCountCI/''' have changed.		Now to see how the implementations have changed, you can just run '''git diff''' to see how the json files in '''unittests/InstructionCountCI/''' have changed.
		+
		+	=== Example ===
		+	Minor improvement when optimizing a move instruction
		+	"movd mm0, eax": {
		+	- "ExpectedInstructionCount": 3,
		+	- "Optimal": "No",
		+	+ "ExpectedInstructionCount": 2,
		+	+ "Optimal": "Yes",
		+	"Comment": "0x0f 0x6e",
		+	"ExpectedArm64ASM": [
		+	- "ubfx x20, x4, #0, #32",
		+	- "fmov s4, w20",
		+	+ "fmov s4, w4",
		+	"str d4, [x28, #752]"
		+	]
		+	},
	* Reset the files with '''git checkout -- unittests/InstructionCountCI/*.json''' if the changes weren't desired.		* Reset the files with '''git checkout -- unittests/InstructionCountCI/*.json''' if the changes weren't desired.