[PATCH] D90717: [llvm] Add a test for debug info generated with split functions.

[Sriraman Tallam via Phabricator via llvm-commits]

tmsriram added inline comments.


================
Comment at: llvm/test/DebugInfo/X86/machine-function-splitter.ll:22
+; CHECK-NEXT: ; /home/llvm-dev/working/llvm-project/build/split_functions.c:7
+; CHECK-NEXT:        0: 01 c0                         addl    %eax, %eax
+; CHECK:      ; /home/llvm-dev/working/llvm-project/build/split_functions.c:9
----------------
What does this correspond to?


================
Comment at: llvm/test/DebugInfo/X86/machine-function-splitter.ll:30
+;;   if (i % 100) {
+;;     return i + 1;
+;;   } else {
----------------
Isn't this the cold basic block? What is split out here?


================
Comment at: llvm/test/DebugInfo/X86/machine-function-splitter.ll:48
+;; llvm-profdata merge -o default.profdata default_*.profraw
+;; clang -fprofile-use -O2 -g -S -emit-llvm split_functions.c
+;; ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ 
----------------
Do we need a -gsplit-dwarf fission test too?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90717