mirror of
https://github.com/clearlinux/clr-avx-tools.git
synced 2026-05-13 10:23:43 +00:00
avxjudge: fix function summary reporting
Refactor the output parsing of `objdump -d ...` so that function summaries are properly reported. Previously, the correspondence of results and functions were off-by-one: the results were mistakenly associated with the next function to parse instead of the previous function that was just parsed. Signed-off-by: Patrick McCarty <patrick.mccarty@intel.com>
This commit is contained in:
committed by
Thiago Macieira
parent
9c58781110
commit
603f8d2354
17
avxjudge.py
17
avxjudge.py
@@ -210,6 +210,18 @@ def process_objdump_line(records:RecordKeeper, line:str, verbose:int, quiet:int)
|
||||
global avx2_avx512_duplicate_cnt
|
||||
global debug
|
||||
|
||||
match = re.search("^$", line)
|
||||
if match:
|
||||
if records.function_record.instructions > 0 and verbose > 0:
|
||||
print()
|
||||
print_function_summary(records)
|
||||
if verbose > 0:
|
||||
print()
|
||||
if records.function_record.instructions > 0:
|
||||
records.finalize_function_attrs()
|
||||
records.function_record = FunctionRecord()
|
||||
return
|
||||
|
||||
match = re.search("^(.*)\#.*", line)
|
||||
if match:
|
||||
line = match.group(1)
|
||||
@@ -230,11 +242,6 @@ def process_objdump_line(records:RecordKeeper, line:str, verbose:int, quiet:int)
|
||||
match = re.search("\<([a-zA-Z0-9_@\.\-]+)\>\:", line)
|
||||
if match:
|
||||
records.function_record.name = match.group(1)
|
||||
if records.function_record.instructions > 0 and verbose > 0:
|
||||
print_function_summary(records)
|
||||
if records.function_record.instructions > 0:
|
||||
records.finalize_function_attrs()
|
||||
records.function_record = FunctionRecord()
|
||||
|
||||
if sse_score >= 0.0:
|
||||
sse_str = str(sse_score)
|
||||
|
||||
Reference in New Issue
Block a user