[Lldb-commits] [PATCH] D52678: DWARFExpression: Resolve file addresses in the linked module
Davide Italiano via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Oct 3 16:27:00 PDT 2018
davide added inline comments.
Comment at: packages/Python/lldbsuite/test/functionalities/target_var/globals.ll:1
+source_filename = "globals.c"
+target datalayout = "e-m:o-i64:64-f80:128-n8:16:32:64-S128"
> Should we check in bitcode instead? That might make it easier to avoid breakage when the textual IR format changes.
But also will be more painful to regenerate in case, e.g. add a verifier check where this breaks.
I think it's a tradeoff anyway, I and Adrian discussed this briefly and agreed this is a decent middle ground (on one extreme, one might check ASM directly, but that seems even harder to handle).
This is honestly based on my experience of having to regenerate broken bitcode files, happy to be convinced otherwise if you have arguments :)
More information about the lldb-commits