yuzu-emu/mbedtls
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

code_size_compare: add comments to make code more readable

Browse source 
Signed-off-by: Yanray Wang <yanray.wang@arm.com>
This commit is contained in:
Yanray Wang 2023-07-25 10:37:11 +08:00
parent 25bd331899
commit e4a3636fac
1 changed files with 15 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
scripts/code_size_compare.py
View file

@ -92,8 +92,9 @@ class CodeSizeDistinctInfo: # pylint: disable=too-few-public-methods
def get_info_indication(self): def get_info_indication(self):
"""Return a unique string to indicate Code Size Distinct Information.""" """Return a unique string to indicate Code Size Distinct Information."""
return '{}-{}-{}-{}'\ return '{rev}-{arch}-{config}-{cc}'\
.format(self.git_rev, self.arch, self.config, self.compiler) .format(rev=self.git_rev, arch=self.arch, config=self.config,
cc=self.compiler)
class CodeSizeCommonInfo: # pylint: disable=too-few-public-methods class CodeSizeCommonInfo: # pylint: disable=too-few-public-methods
@ -112,8 +113,8 @@ class CodeSizeCommonInfo: # pylint: disable=too-few-public-methods
def get_info_indication(self): def get_info_indication(self):
"""Return a unique string to indicate Code Size Common Information.""" """Return a unique string to indicate Code Size Common Information."""
return '{}'\ return '{measure_tool}'\
.format(self.measure_cmd.strip().split(' ')[0]) .format(measure_tool=self.measure_cmd.strip().split(' ')[0])
class CodeSizeResultInfo: # pylint: disable=too-few-public-methods class CodeSizeResultInfo: # pylint: disable=too-few-public-methods
"""Data structure to store result options for code size comparison.""" """Data structure to store result options for code size comparison."""
@ -223,11 +224,11 @@ class CodeSizeBuildInfo: # pylint: disable=too-few-public-methods
"""Infer command to set up proper configuration before running make.""" """Infer command to set up proper configuration before running make."""
pre_make_cmd = [] #type: typing.List[str] pre_make_cmd = [] #type: typing.List[str]
if self.config == SupportedConfig.TFM_MEDIUM.value: if self.config == SupportedConfig.TFM_MEDIUM.value:
pre_make_cmd.append('cp -r {} {}' pre_make_cmd.append('cp -r {src} {dest}'
.format(TFM_MEDIUM_CONFIG_H, CONFIG_H)) .format(src=TFM_MEDIUM_CONFIG_H, dest=CONFIG_H))
pre_make_cmd.append('cp -r {} {}' pre_make_cmd.append('cp -r {src} {dest}'
.format(TFM_MEDIUM_CRYPTO_CONFIG_H, .format(src=TFM_MEDIUM_CRYPTO_CONFIG_H,
CRYPTO_CONFIG_H)) dest=CRYPTO_CONFIG_H))
return pre_make_cmd return pre_make_cmd
@ -641,15 +642,20 @@ class CodeSizeGeneratorWithSize(CodeSizeGenerator):
output.write( output.write(
format_string format_string
.format(fname, .format(fname,
# current(text,data)
str(text_vari[0]) + "," + str(data_vari[0]), str(text_vari[0]) + "," + str(data_vari[0]),
# old(text,data)
str(text_vari[1]) + "," + str(data_vari[1]), str(text_vari[1]) + "," + str(data_vari[1]),
# change(text,data)
str(text_vari[2]) + "," + str(data_vari[2]), str(text_vari[2]) + "," + str(data_vari[2]),
# change%(text,data)
"{:.0%}".format(text_vari[3]) + "," "{:.0%}".format(text_vari[3]) + ","
+ "{:.0%}".format(data_vari[3]))) + "{:.0%}".format(data_vari[3])))
else: else:
output.write( output.write(
format_string format_string
.format(fname, .format(fname,
# current(text,data)
str(text_vari[0]) + "," + str(data_vari[0]), str(text_vari[0]) + "," + str(data_vari[0]),
'None', 'None', 'None')) 'None', 'None', 'None'))