[PATCH] D152817: [MISched] Fix non-debug builds.

Francesco Petrogalli via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 13 07:32:03 PDT 2023


fpetrogalli created this revision.
Herald added subscribers: javed.absar, MatzeB.
Herald added a project: All.
fpetrogalli requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

As reported in https://github.com/llvm/llvm-project/issues/63225, we
need to make sure we can use the `&operator<<` on instances of the
`ResourceSegments` class for builds that set `NDEBUG`.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D152817

Files:
  llvm/include/llvm/CodeGen/MachineScheduler.h


Index: llvm/include/llvm/CodeGen/MachineScheduler.h
===================================================================
--- llvm/include/llvm/CodeGen/MachineScheduler.h
+++ llvm/include/llvm/CodeGen/MachineScheduler.h
@@ -815,7 +815,6 @@
                          const ResourceSegments &c2) {
     return c1._Intervals == c2._Intervals;
   }
-#ifndef NDEBUG
   friend llvm::raw_ostream &operator<<(llvm::raw_ostream &os,
                                        const ResourceSegments &Segments) {
     os << "{ ";
@@ -824,7 +823,6 @@
     os << "}\n";
     return os;
   }
-#endif
 };
 
 /// Each Scheduling boundary is associated with ready queues. It tracks the


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D152817.530902.patch
Type: text/x-patch
Size: 665 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230613/90f41617/attachment.bin>


More information about the llvm-commits mailing list