[PATCH] D69723: [RISCV][RFC] Fix wrong CFI directives

Shiva Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 7 06:36:54 PST 2019


shiva0217 added a comment.

Hi @luismarques,
I tried to generate `.cfi_remember_state` and `.cfi_restore_state` in `emitEpilogue`. It could pass the test case.
But there is an issue that `.cfi_restore_state` will become incorrect if the shrink wrapping hoists the epilogue.
So I think to remove CFI directives from the epilogue as other targets do might be a reasonable step.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69723





More information about the llvm-commits mailing list