[PATCH] D90772: [Coroutines] Add missing llvm.dbg.declare's to cover more allocas

Bruno Cardoso Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 6 10:59:44 PST 2020


bruno added a comment.

> This patch works for llvm.dbg.addr. So what is status of llvm.dbg.addr? does anyone know about it ?

Note sure I follow your comment, did you mean "works for `llvm.dbg.declare`, what about `llvm.dbg.addr`"? My quick code search here seems like the only way to get `llvm.dbg.addr`s are via the llvm option `-use-dbg-addr`. This option doesn't seem to be used by clang (maybe some other frontends do) and using it means changing `getDeclareIntrin()` to give you `.addrs` instead of `.declare`, so I assume this change should work for both?


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

https://reviews.llvm.org/D90772



More information about the llvm-commits mailing list