[PATCH] D87302: [IRSim][IROutliner] Adding DebugInfo handling for IR outlined functions.

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 16 09:58:35 PDT 2020


vsk added a comment.

CodeExtractor contains a utility to fix up debug info after extraction occurs (I believe it's called `fixupDebugInfoPostExtraction`). It's solving the same problem - is there any way to generalize/reuse that logic?



================
Comment at: llvm/test/DebugInfo/AArch64/ir-outliner.ll:2
+; RUN: opt -verify -iroutliner < %s | \
+; RUN: llc -filetype=obj -mtriple=aarch64-- | llvm-dwarfdump - | FileCheck %s
+
----------------
Please validate the debug info metadata in the IR after -iroutliner runs. Validating the final DWARF production is nice, but checking the IR takes precedent (why? well DWARF may not represent everything we care about at the IR level, and DWARF isn't the only debug info format llvm supports).


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

https://reviews.llvm.org/D87302



More information about the llvm-commits mailing list