[PATCH] D125784: [llvm-debuginfo-analyzer] 09 - CodeView Reader

Fri Oct 14 06:57:08 PDT 2022

CarlosAlbertoEnciso added inline comments.

================
Comment at: llvm/lib/DebugInfo/LogicalView/Readers/LVCodeViewVisitor.cpp:3393
+
+Error LVLogicalVisitor::inlineSiteAnnotation(LVScope *AbstractFunction,
+                                             LVScope *InlinedFunction,
----------------
zequanwu wrote:
> zequanwu wrote:
> > It looks like it doesn't handle following case correctly. After parse line 5, it will create line starting at 0x20 with line offset being 3, which should be 2. It's from https://reviews.llvm.org/D123151#3431153. You may want to take a look on https://github.com/llvm/llvm-project/blob/main/llvm/lib/DebugInfo/PDB/Native/NativeInlineSiteSymbol.cpp#L101.
> > ```
> > 0602      line 1 (+1)
> > 0315      code 0x15 (+0x15)
> > 0B2B      code 0x20 (+0xB) line 2 (+1)
> > 0602      line 3 (+1)
> > 0311      code 0x31 (+0x11)
> > 0B29      code 0x3A (+0x9) line 4 (+1)
> > 0409      code end 0x43 (+0x9)
> > ```
> NVM, ignore the comment. It's correct.
Thanks.

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

https://reviews.llvm.org/D125784