[llvm] [llvm-remarkutil] Make invalid states un-representable in the count tool (PR #140829)

Tobias Stadler via llvm-commits llvm-commits at lists.llvm.org
Wed May 28 07:57:59 PDT 2025


================
@@ -250,27 +237,42 @@ Error RemarkCounter::print(StringRef OutputFileName) {
 Expected<Filters> getRemarkFilter() {
   // Create Filter properties.
   std::optional<FilterMatcher> RemarkNameFilter;
+  if (!RemarkNameOpt.empty())
+    RemarkNameFilter = FilterMatcher::createExact(RemarkNameOpt);
+  else if (!RemarkNameOptRE.empty()) {
+    auto FM = FilterMatcher::createRE(RemarkNameOptRE);
+    if (!FM)
+      return FM.takeError();
+    RemarkNameFilter = std::move(*FM);
+  }
+
----------------
tobias-stadler wrote:

We do this a lot, would it be worth adding a `FilterMatcher::createExactOrRE(RemarkNameOpt, RemarkNameOptRE)`? 

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


More information about the llvm-commits mailing list