From ec78fd1876721d3d0ebfbef437f3084ad7b29f23 Mon Sep 17 00:00:00 2001 From: Ramkumar Ramachandra Date: Wed, 12 Jun 2024 10:52:13 +0100 Subject: [PATCH] mlir/Presburger: guard dump function; fix buildbot Follow up on 76030dc (mlir/Presburger/MPInt: move into llvm/ADT) to guard a function in Fraction.h with !NDEBUG || LLVM_ENABLE_DUMP, since the call to the corresponding function in DynamicAPInt is guarded similarly. This patch fixes the build when mlir is built with this configuration. --- .../mlir/Analysis/Presburger/Fraction.h | 23 +++++++++++-------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/mlir/include/mlir/Analysis/Presburger/Fraction.h b/mlir/include/mlir/Analysis/Presburger/Fraction.h index f4f1be97147bf6..6be132058e6caf 100644 --- a/mlir/include/mlir/Analysis/Presburger/Fraction.h +++ b/mlir/include/mlir/Analysis/Presburger/Fraction.h @@ -52,15 +52,24 @@ struct Fraction { return num / den; } - llvm::raw_ostream &print(llvm::raw_ostream &os) const { - return os << "(" << num << "/" << den << ")"; - } - /// The numerator and denominator, respectively. The denominator is always /// positive. DynamicAPInt num{0}, den{1}; + +#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) + llvm::raw_ostream &print(llvm::raw_ostream &os) const { + return os << "(" << num << "/" << den << ")"; + } +#endif }; +#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) +inline llvm::raw_ostream &operator<<(llvm::raw_ostream &os, const Fraction &x) { + x.print(os); + return os; +} +#endif + /// Three-way comparison between two fractions. /// Returns +1, 0, and -1 if the first fraction is greater than, equal to, or /// less than the second fraction, respectively. @@ -156,12 +165,6 @@ inline Fraction &operator*=(Fraction &x, const Fraction &y) { x = x * y; return x; } - -inline llvm::raw_ostream &operator<<(llvm::raw_ostream &os, const Fraction &x) { - x.print(os); - return os; -} - } // namespace presburger } // namespace mlir