[PATCH] D129589: [Attributor] Don't crash if getAnalysisResultForFunction() returns null LoopInfo

Johannes Doerfert via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 12 13:37:54 PDT 2022


jdoerfert accepted this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

LG with return true.



================
Comment at: llvm/lib/Transforms/IPO/AttributorAttributes.cpp:6406
+    if (!LI)
+      return false;
     return LI->getLoopFor(&BB) != nullptr;
----------------
return true is the conservative answer here.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D129589/new/

https://reviews.llvm.org/D129589



More information about the llvm-commits mailing list