From fc129580952d80c2256100dda37492aa3348ef60 Mon Sep 17 00:00:00 2001 From: Ryan Houdek Date: Wed, 16 Aug 2023 13:48:50 -0700 Subject: [PATCH] FEXCore/IR: Fixes bug in IRDumper without specification Didn't notice this in the previous PR, When DUMPIR=stderr without and selection of where to place it in PASSMANAGERDUMPIR it was supposed to put the dumper at the end of the passes. We need to make sure that it it placed at the end of the passes rather than current `it`. --- External/FEXCore/Source/Interface/IR/PassManager.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/External/FEXCore/Source/Interface/IR/PassManager.cpp b/External/FEXCore/Source/Interface/IR/PassManager.cpp index 252307e0d..68e114901 100644 --- a/External/FEXCore/Source/Interface/IR/PassManager.cpp +++ b/External/FEXCore/Source/Interface/IR/PassManager.cpp @@ -60,7 +60,7 @@ void PassManager::Finalize() { if (PassManagerDumpIR() & FEXCore::Config::PassManagerDumpIR::AFTEROPT) { if (!(PassManagerDumpIR() & FEXCore::Config::PassManagerDumpIR::AFTERPASS)) { // Insert final IRDumper. - it = InsertAt(it, Debug::CreateIRDumper()); + InsertAt(Passes.end(), Debug::CreateIRDumper()); } } }