search for: crossregisterproxies

Hi all, I'm trying to build passes with the PassBuilder to optimize the result of MCJIT (I assume, this is a requirement for performance). So I do this: llvm::PassBuilder passBuilder; llvm::ModulePassManager modulePassManager = passBuilder.buildPerModuleDefaultPipeline(llvm::PassBuilder::OptimizationLevel::O3); llvm::ModuleAnalysisManager moduleAnalysisManager;

Hi Philip, Thanks for the response. Under llvm-5.0.2 and llvm-6.0.1 in Debug mode, the crash hit at the same assertion: /usr/local/include/llvm/IR/PassManager.h:689: typename PassT::Result& llvm::AnalysisManager<IRUnitT, ExtraArgTs>::getResult(IRUnitT&, ExtraArgTs ...) [with PassT = llvm::InnerAnalysisManagerProxy<llvm::AnalysisManager<llvm::Loop,

...gerLoopProxy seems to be as simple as: > > /// A proxy from a \c ModuleAnalysisManager to a \c Loop. > typedef OuterAnalysisManagerProxy<ModuleAnalysisManager, Loop, > LoopStandardAnalysisResults &> > ModuleAnalysisManagerLoopProxy; It also needs to be added to PassBuilder::crossRegisterProxies... But yes, that appears to be a required action. regards,   Fedor. > > > > regards, >   Fedor. >> >> >> >> >> On Mon, Mar 4, 2019 at 2:05 PM Fedor Sergeev >> <fedor.sergeev at azul.com <mailto:fedor.sergeev at azul.com>&...

Overall seems fine to me. On 3/11/19 8:12 PM, Hiroshi Yamauchi wrote: > Here's a revised approach based on the discussion: > > - Cache PSI right after the profile summary in the IR is written in > the pass pipeline. This would avoid the need to insert > RequireAnalysisPass for PSI before each non-module pass that needs it. > PSI can be technically invalidated but unlikely