[llvm] [OptBisect] Add support for selecting ranges of passes and refactor DebugCounter to use a shared Range API. (PR #152393)

Nikita Popov via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 5 08:03:28 PDT 2025


================
@@ -0,0 +1,97 @@
+//===- llvm/Support/Range.h - Range parsing utility -----------*- C++ -*-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+//
+// This file provides utilities for parsing range specifications like
+// "1-10,20-30,45" which are commonly used in debugging and bisection tools.
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_SUPPORT_RANGE_H
+#define LLVM_SUPPORT_RANGE_H
+
+#include "llvm/ADT/ArrayRef.h"
+#include "llvm/ADT/SmallVector.h"
+#include "llvm/ADT/StringRef.h"
+#include "llvm/Support/Error.h"
+#include <cstdint>
+
+namespace llvm {
+class raw_ostream;
+} // end namespace llvm
+
+namespace llvm {
+
+/// Represents a range of integers [Begin, End] (inclusive on both ends)
+struct Range {
+  int64_t Begin;
+  int64_t End;
+
+  Range(const int64_t Begin, const int64_t End) : Begin(Begin), End(End) {}
+  Range(const int64_t Single) : Begin(Single), End(Single) {}
+
+  /// Check if the given value is within this range (inclusive)
+  bool contains(const int64_t Value) const {
+    return Value >= Begin && Value <= End;
+  }
+
+  /// Check if this range overlaps with another range
+  bool overlaps(const Range &Other) const {
+    return Begin <= Other.End && End >= Other.Begin;
+  }
+
+  /// Get the size of this range
+  int64_t size() const { return End - Begin + 1; }
+
+  std::string toString() const {
+    return std::to_string(Begin) + "-" + std::to_string(End);
+  }
+
+  void print(raw_ostream &OS) const { OS << Begin << "-" << End; }
+
+  bool operator==(const Range &Other) const {
+    return Begin == Other.Begin && End == Other.End;
+  }
+};
+
+/// Utility class for parsing and managing range specifications
+class RangeUtils {
+public:
+  using RangeList = SmallVector<Range, 8>;
+
+  /// Parse a range specification string like "1-10,20-30,45" or
+  /// "1-10:20-30:45". Ranges must be in increasing order and non-overlapping.
+  /// \param RangeStr The string to parse
+  /// \param Separator The separator character to use (',' or ':')
+  /// \returns Expected<RangeList> containing the parsed ranges on success,
+  ///          or an Error on failure
+  static Expected<RangeList> parseRanges(const StringRef RangeStr,
+                                         const char Separator = ',');
+
+  /// Legacy interface for backward compatibility.
+  /// \deprecated Use the Expected<RangeList> version instead
----------------
nikic wrote:

This is a brand-new API, why have a deprecated interface?

https://github.com/llvm/llvm-project/pull/152393


More information about the llvm-commits mailing list