[llvm] r266446 - [PR27284] Reverse the ownership between DICompileUnit and DISubprogram.
Adrian Prantl via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 15 14:16:45 PDT 2016
> On Apr 15, 2016, at 2:14 PM, Tobias Grosser <tobias at grosser.es> wrote:
>
> On 04/15/2016 10:53 PM, Adrian Prantl wrote:
>> Hi Tobias,
>>
>> there is a python script to upgrade testcases attached to the PR:
>> https://llvm.org/bugs/show_bug.cgi?id=27284
>>
>> Note that it only works for upgrading .ll testcases. If you have source testcases that check for IR, you’ll have to do it manually. The changes are:
>> - DICompileUnit no longer has a list of subprograms:
>> - DISubprogram has a unit: pointing back to the DICompileUnit
>> - The output order of debug metadata is changed because the subprograms no longer are referenced by the CU.
>>
>> let me know if I can help with upgrading the testcases.
>
> Hi Adrian,
>
> your script worked great. I just had to run it once over the test cases (actually tree times as I missed some), and the bugs got fixed. So no further help is needed and the buildbots are green again.
>
> In case you have more such changes in the pipeline, it would be great if you could reference the relevant script from the commit message.
commit 93d9fac22356feb523a37130351dea0d37964212
Author: adrian <adrian at 91177308-0d34-0410-b5e6-96231b3b80d8>
Date: Fri Apr 15 15:57:41 2016 +0000
[PR27284] Reverse the ownership between DICompileUnit and DISubprogram.
<...snip...>
Attached to https://llvm.org/bugs/show_bug.cgi?id=27284 is a python script
that updates LLVM IR testcases to the new format.
http://reviews.llvm.org/D19034
<rdar://problem/25256815>
:-)
> You are obviously also very much invited to just directly change the Polly test cases for such trivial changes without any pre-commit review. If this would be possible, we could probably avoid a couple of buildbot error messages being sent out to random people.
I will try to do this next time.
thanks,
adrian
>
> Best,
> Tobias
>
More information about the llvm-commits
mailing list