diff options
| author | Craig Topper <craig.topper@intel.com> | 2018-12-12 21:21:31 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2018-12-12 21:21:31 +0000 |
| commit | d1c61861ddc94457b08a5a653d3908b7b38ebb22 (patch) | |
| tree | 88e2d24cda24d6d6675a3ace27b4114fb78228a6 /clang/unittests/ASTMatchers/ASTMatchersNodeTest.cpp | |
| parent | 50c9bf45b5ecd913594e68a2b84ee2b9b59936a3 (diff) | |
| download | bcm5719-llvm-d1c61861ddc94457b08a5a653d3908b7b38ebb22.tar.gz bcm5719-llvm-d1c61861ddc94457b08a5a653d3908b7b38ebb22.zip | |
[X86] Don't emit MULX by default with BMI2
MULX has somewhat improved register allocation constraints compared to the legacy MUL instruction. Both output registers are encoded instead of fixed to EAX/EDX, but EDX is used as input. It also doesn't touch flags. Unfortunately, the encoding is longer.
Prefering it whenever BMI2 is enabled is probably not optimal. Choosing it should somehow be a function of register allocation constraints like converting adds to three address. gcc and icc definitely don't pick MULX by default. Not sure what if any rules they have for using it.
Differential Revision: https://reviews.llvm.org/D55565
llvm-svn: 348975
Diffstat (limited to 'clang/unittests/ASTMatchers/ASTMatchersNodeTest.cpp')
0 files changed, 0 insertions, 0 deletions
