SAOC LLDB D integration: 3rd Weekly Update

Johan j at j.nl
Sun Oct 10 17:14:46 UTC 2021


On Wednesday, 6 October 2021 at 22:34:41 UTC, Luís Ferreira wrote:
> Hi D community!
>
> I'm here again, to describe what I've done during the third 
> week of Symmetry Autumn of Code.
>
> ## Restructuring the patch history
>
> According to LLVM team, the 
> [patch](https://reviews.llvm.org/D110576) I previously sent 
> needs to be split into smaller ones and that is a bit of a 
> challenge. Why you may ask? Because looking at the [ABI 
> specification](https://dlang.org/spec/abi.html) a lot of 
> mangling nodes are dependent of each other and splitting 3000 
> lines of code into small, stackable and independent patches 
> needs some planning. I tried several layouts and 
> [this](https://pad.riseup.net/p/r.d96e7d99b8d964cca079be42c27e3656) one seems to work out the best, so I'm sticking with it.
>
> Unfortunately, this week was kinda unproductive as I mostly 
> spent this week entirely restructuring the patch.

Hi Luis,
   It's nice to read your updates and see that you are not put off 
by the high quality standard in the LLVM project.
The way I see it, your week was not unproductive at all, 
because...

> ## Fixing a bug on libiberty D demangler
>
> During the restructure, I found out that anonymous symbols were 
> not correctly handled and skipped, according to the 
> specification

... pursuing the smaller patch / more testcase path let you find 
a bug :)

cheers,
   Johan



More information about the Digitalmars-d mailing list