Skip to content

[X86] Split WriteIMul into 8/16/32/64 implementations #36279

@RKSimon

Description

@RKSimon
Bugzilla Link 36931
Resolution FIXED
Resolved on Sep 24, 2018 08:57
Version trunk
OS Windows NT
Depends On #35432
Blocks #31672 #35898 #36479 #36256
CC @alexey-bataev,@adibiagio,@legrosbuffle,@topperc,@rotateright
Fixed by commit(s) rL331770, rL342892

Extended Description

MUL64 definitely needs its own scheduler class, possibly the others as well. Multiply by constant (IMUL32rmi etc.) may need splitting off as well.

Note: Integer multiplies seem to be a common issue in vectorization regressions (incorrectly comparing scalar/vector integer multiply costs), so all need to be accurate for [Bug #​36550] to work well.

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions