[BOLT] Remove workaround

Remove the workaround for the issue that was fixed in #160263.

Author: bgergely0

bolt/lib/Passes/InsertNegateRAStatePass.cpp

@@ -126,26 +126,15 @@ Error InsertNegateRAState::runOnFunctions(BinaryContext &BC) {
     return !BF.containedNegateRAState() || BF.isIgnored();
   };
 
-  int Total = llvm::count_if(
-      BC.getBinaryFunctions(),
-      [&](std::pair<const unsigned long, BinaryFunction> &P) {
-        return P.second.containedNegateRAState() && !P.second.isIgnored();
-      });
-
-  // If we attempt to run the pass, but SkipPredicate skips all functions,
-  // we get a warning that the SchedulingPolicy is changed to trivial.
-  // This would break bolt/test/AArch64/unmarked-data.test.
-  if (Total != 0) {
-    ParallelUtilities::runOnEachFunction(
-        BC, ParallelUtilities::SchedulingPolicy::SP_INST_LINEAR, WorkFun,
-        SkipPredicate, "InsertNegateRAStatePass");
-
-    BC.outs() << "BOLT-INFO: rewritten pac-ret DWARF info in "
-              << FunctionsModified << " out of "
-              << BC.getBinaryFunctions().size() << " functions "
-              << format("(%.2lf%%).\n", (100.0 * FunctionsModified) /
-                                            BC.getBinaryFunctions().size());
-  }
+  ParallelUtilities::runOnEachFunction(
+      BC, ParallelUtilities::SchedulingPolicy::SP_INST_LINEAR, WorkFun,
+      SkipPredicate, "InsertNegateRAStatePass");
+
+  BC.outs() << "BOLT-INFO: rewritten pac-ret DWARF info in "
+            << FunctionsModified << " out of " << BC.getBinaryFunctions().size()
+            << " functions "
+            << format("(%.2lf%%).\n", (100.0 * FunctionsModified) /
+                                          BC.getBinaryFunctions().size());
   return Error::success();
 }
 

bolt/lib/Passes/MarkRAStates.cpp

@@ -137,18 +137,13 @@ Error MarkRAStates::runOnFunctions(BinaryContext &BC) {
         return P.second.containedNegateRAState() && !P.second.isIgnored();
       });
 
-  // If we attempt to run the pass, but SkipPredicate skips all functions,
-  // we get a warning that the SchedulingPolicy is changed to trivial.
-  // This would break bolt/test/AArch64/unmarked-data.test.
-  if (Total != 0) {
-    ParallelUtilities::runOnEachFunction(
-        BC, ParallelUtilities::SchedulingPolicy::SP_INST_LINEAR, WorkFun,
-        SkipPredicate, "MarkRAStates");
-    BC.outs() << "BOLT-INFO: MarkRAStates ran on " << Total
-              << " functions. Ignored " << FunctionsIgnored << " functions "
-              << format("(%.2lf%%)", (100.0 * FunctionsIgnored) / Total)
-              << " because of CFI inconsistencies\n";
-  }
+  ParallelUtilities::runOnEachFunction(
+      BC, ParallelUtilities::SchedulingPolicy::SP_INST_LINEAR, WorkFun,
+      SkipPredicate, "MarkRAStates");
+  BC.outs() << "BOLT-INFO: MarkRAStates ran on " << Total
+            << " functions. Ignored " << FunctionsIgnored << " functions "
+            << format("(%.2lf%%)", (100.0 * FunctionsIgnored) / Total)
+            << " because of CFI inconsistencies\n";
 
   return Error::success();
 }