Include bound typevars in the instruction predicate.

We already do this for the encoding tables, but the instruction
predicates computed by Apply.inst_predicate() did not include them.

Make sure we don't duplicate the type check in the Encoding constructor
when passed an Apply AST node.
This commit is contained in:
Jakob Stoklund Olesen
2017-07-28 14:52:24 -07:00
parent 9df0b09301
commit e2bf4f8981
2 changed files with 18 additions and 10 deletions

View File

@@ -7,7 +7,7 @@ for patern matching an rewriting of cretonne instructions.
from __future__ import absolute_import
from . import instructions
from .typevar import TypeVar
from .predicates import IsEqual, And
from .predicates import IsEqual, And, TypePredicate
try:
from typing import Union, Tuple, Sequence, TYPE_CHECKING, Dict, List # noqa
@@ -367,6 +367,13 @@ class Apply(Expr):
pred = And.combine(pred, IsEqual(ffield, arg))
# Add checks for any bound type variables.
for bound_ty, tv in zip(self.typevars, self.inst.all_typevars()):
if bound_ty is None:
continue
type_chk = TypePredicate.typevar_check(self.inst, tv, bound_ty)
pred = And.combine(pred, type_chk)
return pred
def copy(self, m):