fix: remove incorrect CPP guard around hscCheckSafeImports

Safe Haskell import checking (hscCheckSafeImports) is a compile-time
feature that should work regardless of interpreter support. The previous
CPP guard `#if !defined(HAVE_INTERPRETER)` caused stage1 builds (which
use -interpreter flag) to skip Safe Haskell checking when compiling
boot libraries.

This resulted in incorrect Safe Haskell metadata in .hi files, causing
testsuite failures for Safe Haskell tests (BadImport, Dep, ImpSafe,
safePkg01, etc.) because ghc-internal modules compiled by stage1 had
different safety inferences than expected.

The fix removes the CPP guard so hscCheckSafeImports always runs,
ensuring consistent Safe Haskell behavior regardless of whether the
compiler has interpreter support enabled.

Author: angerman

compiler/GHC/Driver/Main.hs

@@ -771,11 +771,7 @@ tcRnModule' sum save_rn_syntax mod = do
 
       -- module (could be) safe, throw warning if needed
       else do
-#if !defined(HAVE_INTERPRETER)
-          let tcg_res' = tcg_res
-#else
           tcg_res' <- hscCheckSafeImports tcg_res
-#endif
           safe <- liftIO $ readIORef (tcg_safe_infer tcg_res')
           when safe $
             case wopt Opt_WarnSafe dflags of