[PATCH] D78778: Add SupportsDebugUnwindInformation to MCAsmInfo

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 29 07:57:49 PDT 2020


probinson added a comment.

I haven't touched the streamer side of AsmPrinter in a long time, but...
On PS4 we default to `-fno-exceptions` and while the section ends up named `.eh_frame` instead of `.debug_frame` the content is fine; we've had no complaints from our debugger folks.  Aside from the introduction of the .cfi directives (which you do want), I'm not aware that there's any codegen consequence to this combination.
So, I'm not clear why a special streamer is really necessary.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78778



More information about the llvm-commits mailing list