[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