Optimize Memory Region Search with Break

luzhixing12345 · sjp38 · commit 1349e0f571e0 · 2024-09-25T07:44:43.000-07:00
A pattern corresponds to only one area of memory region.
So if match the corresponding region name, there's no
need to check for others.

Signed-off-by: luzhixing12345 &lt;luzhixing12345@163.com&gt;
diff --git a/masim.c b/masim.c
@@ -513,6 +513,7 @@ int parse_phase(char *lines[], int nr_lines, struct phase *p,
 		for (k = 0; k < nr_regions; k++) {
 			if (strcmp(fields[0], regions[k].name) == 0) {
 				a->mregion = &regions[k];
+				break;
 			}
 		}
 		if (a->mregion == NULL)

Original file line number	Diff line number	Diff line change
`@@ -513,6 +513,7 @@ int parse_phase(char lines[], int nr_lines, struct phase p,`
`513`	`513`	`for (k = 0; k < nr_regions; k++) {`
`514`	`514`	`if (strcmp(fields[0], regions[k].name) == 0) {`
`515`	`515`	`a->mregion = &regions[k];`
	`516`	`+ break;`
`516`	`517`	`}`
`517`	`518`	`}`
`518`	`519`	`if (a->mregion == NULL)`