[PATCH] D146543: [Coroutines] Look for dbg.declare for temp spills

Chuanqi Xu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 28 01:27:25 PDT 2023


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

Oh, you're right. It is indeed easier to handle them in the early place.



================
Comment at: llvm/lib/Transforms/Coroutines/CoroFrame.cpp:1852
+        auto *CurDef = Def;
+        while (DIs.empty() && isa<LoadInst>(CurDef)) {
+          auto *LdInst = cast<LoadInst>(CurDef);
----------------
Let's add a fast check here to avoid fall in loop if the program is not compiled with debug information.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146543



More information about the llvm-commits mailing list