[GR-29304] [GR-30676] Do not check hasPendingSideEffectingActions() in runPendingSafepointActions()

* Notably for "after" it was incorrect since at that point we already
  restored the allowSideEffects value, and then it would always return
  false if prevSideEffects is true.

Author: eregon

src/main/java/org/truffleruby/core/thread/ThreadNodes.java

@@ -341,7 +341,7 @@ protected Object handleInterrupt(RubyThread self, RubySymbol timing, RubyProc bl
             try {
                 if (newInterruptMode == InterruptMode.IMMEDIATE) {
                     beforeProfile.enter();
-                    runPendingSafepointActions(safepoint, "before");
+                    runPendingSafepointActions("before");
                 }
 
                 return callBlock(block);
@@ -351,20 +351,17 @@ protected Object handleInterrupt(RubyThread self, RubySymbol timing, RubyProc bl
 
                 if (oldInterruptMode != InterruptMode.NEVER) {
                     afterProfile.enter();
-                    runPendingSafepointActions(safepoint, "after");
+                    runPendingSafepointActions("after");
                 }
             }
         }
 
         @TruffleBoundary
-        private void runPendingSafepointActions(TruffleSafepoint safepoint, String when) {
-            if (safepoint.hasPendingSideEffectingActions()) {
-                if (getContext().getOptions().LOG_PENDING_INTERRUPTS) {
-                    RubyLanguage.LOGGER
-                            .info("Running pending interrupts " + when + " Thread.handle_interrupt");
-                }
-                TruffleSafepoint.pollHere(this);
+        private void runPendingSafepointActions(String when) {
+            if (getContext().getOptions().LOG_PENDING_INTERRUPTS) {
+                RubyLanguage.LOGGER.info("Running pending interrupts " + when + " Thread.handle_interrupt");
             }
+            TruffleSafepoint.pollHere(this);
         }
 
         private InterruptMode symbolToInterruptMode(RubyLanguage language, RubySymbol symbol) {