[DirectX] fix crash in passes when building with LLVM_ENABLE_EXPENSIVE_CHECKS

fixes #148681

For the scalarizer pass we just need to indicate that scalarization took
place, I used the logic for knowing when to eraseFromParent to indicate
this.

For the DXILLegalizePass and DXILResourceAccess.cpp preserve all seems
to be the wrong behavior so stopped doing that.

Author: farzonl

llvm/lib/Target/DirectX/DXILLegalizePass.cpp

@@ -629,7 +629,6 @@ class DXILLegalizationPipeline {
 
       for (auto *Inst : reverse(ToRemove))
         Inst->eraseFromParent();
-
       MadeChange |= !ToRemove.empty();
     }
     return MadeChange;
@@ -676,16 +675,16 @@ class DXILLegalizeLegacy : public FunctionPass {
 PreservedAnalyses DXILLegalizePass::run(Function &F,
                                         FunctionAnalysisManager &FAM) {
   DXILLegalizationPipeline DXLegalize;
-  bool MadeChanges = DXLegalize.runLegalizationPipeline(F);
-  if (!MadeChanges)
-    return PreservedAnalyses::all();
-  PreservedAnalyses PA;
-  return PA;
+  DXLegalize.runLegalizationPipeline(F);
+  // Note: Nothing seems to need Legalization to preserve analysis.
+  // Simplest fix was to not preserve analysis.
+  return PreservedAnalyses::none();
 }
 
 bool DXILLegalizeLegacy::runOnFunction(Function &F) {
   DXILLegalizationPipeline DXLegalize;
-  return DXLegalize.runLegalizationPipeline(F);
+  DXLegalize.runLegalizationPipeline(F);
+  return true;
 }
 
 char DXILLegalizeLegacy::ID = 0;

llvm/lib/Target/DirectX/DXILResourceAccess.cpp

@@ -264,10 +264,10 @@ PreservedAnalyses DXILResourceAccess::run(Function &F,
       MAMProxy.getCachedResult<DXILResourceTypeAnalysis>(*F.getParent());
   assert(DRTM && "DXILResourceTypeAnalysis must be available");
 
-  bool MadeChanges = transformResourcePointers(F, *DRTM);
-  if (!MadeChanges)
-    return PreservedAnalyses::all();
+  transformResourcePointers(F, *DRTM);
 
+  // Note: Regardless of transformation only preserve
+  // DXILResourceTypeAnalysis and DominatorTreeAnalysis.
   PreservedAnalyses PA;
   PA.preserve<DXILResourceTypeAnalysis>();
   PA.preserve<DominatorTreeAnalysis>();
@@ -281,7 +281,10 @@ class DXILResourceAccessLegacy : public FunctionPass {
     DXILResourceTypeMap &DRTM =
         getAnalysis<DXILResourceTypeWrapperPass>().getResourceTypeMap();
 
-    return transformResourcePointers(F, DRTM);
+    transformResourcePointers(F, DRTM);
+    // Note: Always returning true ensures the pass manager knows
+    // we've potentially modified the IR
+    return true;
   }
   StringRef getPassName() const override { return "DXIL Resource Access"; }
   DXILResourceAccessLegacy() : FunctionPass(ID) {}

llvm/lib/Transforms/Scalar/Scalarizer.cpp

@@ -458,8 +458,11 @@ bool ScalarizerVisitor::visit(Function &F) {
       Instruction *I = &*II;
       bool Done = InstVisitor::visit(I);
       ++II;
-      if (Done && I->getType()->isVoidTy())
+      if (Done && I->getType()->isVoidTy()) {
         I->eraseFromParent();
+        // Explicitly mark that we've modified the IR
+        Scalarized = true;
+      }
     }
   }
   return finish();
@@ -1334,7 +1337,6 @@ bool ScalarizerVisitor::finish() {
   Scalarized = false;
 
   RecursivelyDeleteTriviallyDeadInstructionsPermissive(PotentiallyDeadInstrs);
-
   return true;
 }