[PATCH] D78778: Add SupportsDebugUnwindInformation to MCAsmInfo

[Venkata Ramanaiah Nalamothu via Phabricator via llvm-commits]

RamNalamothu added inline comments.


================
Comment at: llvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp:328
 
+  if (MMI->hasDebugInfo() &&
+      MAI->getExceptionHandlingType() == ExceptionHandling::None &&
----------------
scott.linder wrote:
> RamNalamothu wrote:
> > This ignores //ForceDwarfFrameSection// which is independent of //MMI->hasDebugInfo()//.
> `ForceDwarfFrameSection` has no effect when `MAI->getExceptionHandlingType() == ExceptionHandling::None` though, so I believe this condition implicity respects it? The semantics of `MAI->doesSupportDebugUnwindInformation()` subsume `ForceDwarfFrameSection` anyway as it always produces `.debug_frame`.
`ForceDwarfFrameSection` is independent otherwise there is the point in saying force something and this reflects in `MachineFunction::needsFrameMoves()`.

Yes, for AMDGPU we do get `.debug_frame` always, but this piece of code is generic and not specific to AMDGPU.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78778