diff options
| author | Craig Topper <craig.topper@intel.com> | 2018-06-27 00:42:24 +0000 |
|---|---|---|
| committer | Craig Topper <craig.topper@intel.com> | 2018-06-27 00:42:24 +0000 |
| commit | 33aba0eb4c7e582577355659a593eef1ddcefee8 (patch) | |
| tree | 158c05aa21d89d698519266f7e20337cdc6ae919 /llvm/lib/Target/X86/X86InstrInfo.cpp | |
| parent | 54172257458f1e04f68660391d0cba93492c479a (diff) | |
| download | bcm5719-llvm-33aba0eb4c7e582577355659a593eef1ddcefee8.tar.gz bcm5719-llvm-33aba0eb4c7e582577355659a593eef1ddcefee8.zip | |
[X86] Don't store register and memory FMA3 opcodes in the same X86InstrFMA3Group.
Nothing was using this relationship. By splitting them we no longer need to worry about register or memory entries being empty in a group.
The memory folding tables in X86InstrInfo.cpp can be used to access this relationship if needed.
llvm-svn: 335694
Diffstat (limited to 'llvm/lib/Target/X86/X86InstrInfo.cpp')
| -rw-r--r-- | llvm/lib/Target/X86/X86InstrInfo.cpp | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/llvm/lib/Target/X86/X86InstrInfo.cpp b/llvm/lib/Target/X86/X86InstrInfo.cpp index 94d381a1901..a5c6b19dd0a 100644 --- a/llvm/lib/Target/X86/X86InstrInfo.cpp +++ b/llvm/lib/Target/X86/X86InstrInfo.cpp @@ -6766,15 +6766,9 @@ unsigned X86InstrInfo::getFMA3OpcodeToCommuteOperands( }; unsigned FMAForms[3]; - if (FMA3Group.isRegOpcodeFromGroup(Opc)) { - FMAForms[0] = FMA3Group.getReg132Opcode(); - FMAForms[1] = FMA3Group.getReg213Opcode(); - FMAForms[2] = FMA3Group.getReg231Opcode(); - } else { - FMAForms[0] = FMA3Group.getMem132Opcode(); - FMAForms[1] = FMA3Group.getMem213Opcode(); - FMAForms[2] = FMA3Group.getMem231Opcode(); - } + FMAForms[0] = FMA3Group.get132Opcode(); + FMAForms[1] = FMA3Group.get213Opcode(); + FMAForms[2] = FMA3Group.get231Opcode(); unsigned FormIndex; for (FormIndex = 0; FormIndex < 3; FormIndex++) if (Opc == FMAForms[FormIndex]) |
