[PATCH] D43219: [PartialInlining] Fix Crash from holding a reference to a destructed ORE
Graham Yiu via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Mar 20 10:49:21 PDT 2018
graham-yiu-huawei added a comment.
@sfertile Sorry for the late response. AFAIK there shouldn't be a downside to creating the ORE objects at the point of use, I only did it the original way to conform to the style I saw in other passes and to avoid passing the ORE object from function to function. However, my knowledge in the ORE area is limited so someone else may contradict what I just said. Good luck!
Repository:
rL LLVM
https://reviews.llvm.org/D43219
More information about the llvm-commits
mailing list