diff options
| author | Andrew Trick <atrick@apple.com> | 2013-09-25 18:14:12 +0000 |
|---|---|---|
| committer | Andrew Trick <atrick@apple.com> | 2013-09-25 18:14:12 +0000 |
| commit | 070156437752179833b1e5fddd50caa03fd7c12f (patch) | |
| tree | 51f3737749c059861d132cc0390b8693fb908136 /lib/CodeGen/TargetSchedule.cpp | |
| parent | 2558c2bfbaa4bfd49ee8e06be78a0ccb0f3ff0e7 (diff) | |
| download | external_llvm-070156437752179833b1e5fddd50caa03fd7c12f.tar.gz external_llvm-070156437752179833b1e5fddd50caa03fd7c12f.tar.bz2 external_llvm-070156437752179833b1e5fddd50caa03fd7c12f.zip | |
Mark the x86 machine model as incomplete. PR17367.
Ideally, the machinel model is added at the time the instructions are
defined. But many instructions in X86InstrSSE.td still need a model.
Without this workaround the scheduler asserts because x86 already has
itinerary classes for these instructions, indicating they should be
modeled by the scheduler. Since we use the new machine model for other
instructions, it expects a new machine model for these too.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@191391 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/TargetSchedule.cpp')
| -rw-r--r-- | lib/CodeGen/TargetSchedule.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/CodeGen/TargetSchedule.cpp b/lib/CodeGen/TargetSchedule.cpp index 64ee9d1c46..fd3f49657b 100644 --- a/lib/CodeGen/TargetSchedule.cpp +++ b/lib/CodeGen/TargetSchedule.cpp @@ -210,7 +210,8 @@ unsigned TargetSchedModel::computeOperandLatency( // unit latency (defaultDefLatency may be too conservative). #ifndef NDEBUG if (SCDesc->isValid() && !DefMI->getOperand(DefOperIdx).isImplicit() - && !DefMI->getDesc().OpInfo[DefOperIdx].isOptionalDef()) { + && !DefMI->getDesc().OpInfo[DefOperIdx].isOptionalDef() + && SchedModel.isComplete()) { std::string Err; raw_string_ostream ss(Err); ss << "DefIdx " << DefIdx << " exceeds machine model writes for " |