[PATCH] D96393: [CodeGen] Add Machine Function Splitter and DebugInfo composite testcase

[Snehasish Kumar via Phabricator via llvm-commits]

snehasish added a comment.

We have discussed this approach and decided against adding a separate debug info test for function splitting (see the discussion in D90717 <https://reviews.llvm.org/D90717>). The underlying functionality is tested in test/DebugInfo/X86/basic-block-sections_1.ll. Based on the direction in your previous patch (D95209 <https://reviews.llvm.org/D95209>) - emitting dwarf based debug info for PE/COFF executables has limited utility as far as I can tell. While tooling under cygwin will be able to use this information most Windows native debuggers may not support reading from dwarf (eg. libdwarf started supporting PE mid-2019: https://www.prevanders.net/dwarf.html#libdwarf-20190104). For Windows, we need to look into the CodeView format (see the slides from https://llvm.org/devmtg/2016-11/Slides/Kleckner-CodeViewInLLVM.pdf).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96393