[PATCH] D40979: [DWARF] Allow duplication of tails with CFI instructions

Djordje Kovacevic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 30 08:41:35 PST 2018


djokov added a comment.

In https://reviews.llvm.org/D40979#991408, @thegameg wrote:

> If we end up duplicating a prologue like in the test case that r317726 broke, will the CFI be correct without Correct dwarf unwind information in function epilogue <https://reviews.llvm.org/D35844>?


They will not be completely correct (one correcting .cfi_def_cfa_offset instruction on the beginning of the duplicated block would be missing), but hopefully this won't be a problem, because it is planned to reland Correct dwarf unwind information in function epilogue <https://reviews.llvm.org/D35844> right after this patch.


Repository:
  rL LLVM

https://reviews.llvm.org/D40979





More information about the llvm-commits mailing list