[LLVMdev] How do I update Ocaml debug info? (was Re: [llvm] r218914 - DI: Fold constant arguments into a single MDString)
Duncan P. N. Exon Smith
dexonsmith at apple.com
Thu Oct 2 16:20:12 PDT 2014
> On Oct 2, 2014, at 4:17 PM, Peter Zotov <whitequark at whitequark.org> wrote:
> On 2014-10-03 03:16, Duncan P. N. Exon Smith wrote:
>>> On Oct 2, 2014, at 3:48 PM, Duncan P. N. Exon Smith <dexonsmith at apple.com> wrote:
>>> -llvm-commits, +llvmdev
>>>> On Oct 2, 2014, at 2:57 PM, Duncan P. N. Exon Smith <dexonsmith at apple.com> wrote:
>>>> Author: dexonsmith
>>>> Date: Thu Oct 2 16:56:57 2014
>>>> New Revision: 218914
>>>> URL: http://llvm.org/viewvc/llvm-project?rev=218914&view=rev
>>>> DI: Fold constant arguments into a single MDString
>>>> This patch addresses the first stage of PR17891 by folding constant
>>>> arguments together into a single MDString. Integers are stringified and
>>>> a `\0` character is used as a separator.
>>>> Part of PR17891.
>>>> Note: I've attached my testcases upgrade scripts to the PR. If I've
>>>> just broken your out-of-tree testcases, they might help.
>>> Among other things, this commit broke the Ocaml debug info test .
>>> It's not obvious to me how to fix it (I don't speak Ocaml). Anyone know
>>> what to do? (Is it reasonable for me to just XFAIL it, file a PR, and
>>> move on?)
>>> : http://lab.llvm.org:8011/builders/llvm-hexagon-elf/builds/11579
>> I have a guess (patch below), which I'll roll into my next attempt once
>> I've sorted out the other buildbot failures. Note that r218914 had the
>> first hunk, but missed the second.
>> In the meantime, if anyone that speaks Ocaml can take a look, I'd
>> appreciate it.
Awesome, thanks. I'll roll that into my next attempt.
More information about the llvm-dev