Skip to content

[X86] Remove Schedule classes from models that don't use them #34956

@RKSimon

Description

@RKSimon
Bugzilla Link 35608
Resolution FIXED
Resolved on Aug 09, 2018 06:22
Version trunk
OS Windows NT
Blocks #31672 #36479 #36256
CC @adibiagio,@legrosbuffle,@topperc
Fixed by commit(s) 334389

Extended Description

We currently have to specify all the InstrSchedModel classes even if the model doesn't support it (e.g. WriteVarVecShift on pre-AVX2 machines).

We should be able to not have to include these as long as the model's UnsupportedFeatures list agrees. Naturally if we try to attach the model to a cpu that does support that feature we should error out (e.g. X86ScheduleHaswell being used on KNL).

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions