[patch] migrating DragonEgg to DIBuilder

Duncan Sands baldrick at free.fr
Sat Feb 23 09:33:35 PST 2013


Hi David,

>     Your patch still applies, and now the only failure is the objc member problem
>     I mentioned before.
>
>
> Looking at this, it seems like there's no change in behavior - the test appears
> to have been accidentally passing because the IR dumper wasn't printing out the
> annotated debug info comments next to the metadata (I guess the IR printer lost
> support for annotating the old debug info format somewhere along the way)
>
> Attached are the old (ToT) & new (with my patch applied) IR for this test case -
> perhaps I'm missing something about what the test case is meant to test, etc.

it's hard for me to comment since I have no idea what this metadata means, and
why dragonegg is now outputting so much more than clang and the old dragonegg.
For example I don't know what you mean by "annotated debug info comments".

Ciao, Duncan.




More information about the llvm-commits mailing list