================ @@ -3280,7 +3280,7 @@ class llvm::gvn::GVNLegacyPass : public FunctionPass { if (skipFunction(F)) return false; - auto *LIWP = getAnalysisIfAvailable<LoopInfoWrapperPass>(); + auto &LIWP = getAnalysis<LoopInfoWrapperPass>(); ---------------- jayfoad wrote:
I prefer not to leave it as is - I think either we should get the analysis with `getAnalysis` or we should remove the `addRequired`, so that they are consistent with each other. https://github.com/llvm/llvm-project/pull/65729 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits