[PATCH] D97673: [RFC] [[Coroutine] [Debug] Salvage dbg.values

[Chuanqi Xu via Phabricator via llvm-commits]

ChuanqiXu added inline comments.


================
Comment at: llvm/lib/Transforms/Coroutines/CoroFrame.cpp:2305-2308
+  for (auto *V : FrameData.getAllDefs()) {
+    // We would handle alloca specially.
+    if (isa<AllocaInst>(V))
+      continue;
----------------
lxfind wrote:
> This is basically iterating over `FrameData.Spills` and obtain `.first` on the iterator
Yes, I am just wondering if the code now makes the semantics more clear. If we write
```
for (auto *Iter : FrameData.Spills) {
     auto *V = Iter.first;
     // ...
}
```
we need to comment that we would handle alloca specially. And the style tells it in the level of code.


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

https://reviews.llvm.org/D97673