Fix bug introduced in r196517.
authorDaniel Jasper <djasper@google.com>
Fri, 6 Dec 2013 08:58:22 +0000 (08:58 +0000)
committerDaniel Jasper <djasper@google.com>
Fri, 6 Dec 2013 08:58:22 +0000 (08:58 +0000)
Not only does it trigger -Wparentheses, I think the assert actually
relies on incorrect operator precedence.

Also, the grammar as questionable, but I might not know enough about the
problem at hand.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@196567 91177308-0d34-0410-b5e6-96231b3b80d8

lib/CodeGen/MachineScheduler.cpp

index 6cfedcbbc2e7ab5779aea022ff51a10205c7c40c..15035967be32bdc0f99414b8cad29f654bb6529b 100644 (file)
@@ -2175,8 +2175,9 @@ void GenericScheduler::SchedBoundary::bumpNode(SUnit *SU) {
   // exceed the issue width.
   const MCSchedClassDesc *SC = DAG->getSchedClass(SU);
   unsigned IncMOps = SchedModel->getNumMicroOps(SU->getInstr());
-  assert(CurrMOps == 0 || (CurrMOps + IncMOps) <= SchedModel->getIssueWidth() &&
-         "Cannot scheduling this instructions MicroOps in the current cycle.");
+  assert(
+      (CurrMOps == 0 || (CurrMOps + IncMOps) <= SchedModel->getIssueWidth()) &&
+      "Cannot schedule this instructions MicroOps in the current cycle.");
 
   unsigned ReadyCycle = (isTop() ? SU->TopReadyCycle : SU->BotReadyCycle);
   DEBUG(dbgs() << "  Ready @" << ReadyCycle << "c\n");