diff options
author | Guido van Rossum <guido@python.org> | 2023-07-18 01:30:41 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-07-18 01:30:41 (GMT) |
commit | 1e36ca63f9f5e0399efe13a80499cef290314c2a (patch) | |
tree | 4e5ddbcd299a65c5f8eb90cd70ff26b62c4b63a4 /Tools | |
parent | 00e52acebd2beb2663202bfc4be0ce79ba77361e (diff) | |
download | cpython-1e36ca63f9f5e0399efe13a80499cef290314c2a.zip cpython-1e36ca63f9f5e0399efe13a80499cef290314c2a.tar.gz cpython-1e36ca63f9f5e0399efe13a80499cef290314c2a.tar.bz2 |
Small fixes to code generator (#106845)
These repair nits I found in PR gh-106798 (issue gh-106797) and in PR gh-106716 (issue gh-106706).
Diffstat (limited to 'Tools')
-rw-r--r-- | Tools/cases_generator/generate_cases.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/Tools/cases_generator/generate_cases.py b/Tools/cases_generator/generate_cases.py index 037bee1..2713fc6 100644 --- a/Tools/cases_generator/generate_cases.py +++ b/Tools/cases_generator/generate_cases.py @@ -99,7 +99,7 @@ def effect_size(effect: StackEffect) -> tuple[int, str]: return 0, effect.size elif effect.cond: if effect.cond in ("0", "1"): - return 0, effect.cond + return int(effect.cond), "" return 0, f"{maybe_parenthesize(effect.cond)} ? 1 : 0" else: return 1, "" @@ -841,9 +841,9 @@ class Analyzer: def check_families(self) -> None: """Check each family: - - Must have at least 2 members - - All members must be known instructions - - All members must have the same cache, input and output effects + - Must have at least 2 members (including head) + - Head and all members must be known instructions + - Head and all members must have the same cache, input and output effects """ for family in self.families.values(): if family.name not in self.macro_instrs and family.name not in self.instrs: @@ -868,7 +868,7 @@ class Analyzer: self.error( f"Family {family.name!r} has inconsistent " f"(cache, input, output) effects:\n" - f" {family.members[0]} = {expected_effects}; " + f" {family.name} = {expected_effects}; " f"{member} = {member_effects}", family, ) |