[PATCH] D73383: [Debuginfo] Allow retrieving source files relative to the compilation directory.

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: llvm/unittests/DebugInfo/DWARF/DWARFDebugLineTest.cpp:909
+  LineTable &LT = Gen->addLineTable();
+  LT.setCustomPrologue({
+      {33, LineTable::Long},   // unit length
----------------
jhenderson wrote:
> saugustine wrote:
> > jhenderson wrote:
> > > Rather than needing to create an entire custom prologue, you can use `createBasicPrologue` to generate a default one, and then just tweak the contents to suit your needs (not forgetting to update the length fields in the process). See `ErrorForUnitLengthTooLarge` for an example usage.
> > > 
> > > This will improve readability as it will more clearly show what is important to to this test.
> > Doing this required modifying how basic prologues are created, which adds a few changes in random places. I had proposed D74249 separately, but the reviewer recommended I roll that into this.
> As noted in D74249, you can avoid folding it in, by extending the `checkDefaultPrologue` checks.
To avoid more churn in review, etc, I went ahead and committed D74249 with the suggested change to ```checkDefaultPrologue``` ( in 0bd48c3d4ee1a94ea3d3b9d89201b23fd83c94d0 ) - so should be able to sync/resolve this patch with that and move forward from there.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D73383