[PATCH] D137201: [AggressiveInstCombine] Handle the insert point of the merged load correctly.
Alina Sbirlea via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 2 11:12:47 PDT 2022
asbirlea added inline comments.
================
Comment at: llvm/lib/Transforms/AggressiveInstCombine/AggressiveInstCombine.cpp:783
+ LOps.InsertPoint = Start;
+ } else if (LOps.InsertPoint && Start->comesBefore(LOps.InsertPoint))
+ LOps.InsertPoint = Start;
----------------
Is it possible for InsertPoint to be `nullptr` here?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D137201/new/
https://reviews.llvm.org/D137201
More information about the llvm-commits
mailing list