[llvm-commits] Add line numbers to OProfile
clattner at apple.com
Wed Jul 15 20:16:12 CDT 2009
On Jul 15, 2009, at 4:01 PM, Jeffrey Yasskin wrote:
> On Wed, Jul 15, 2009 at 3:57 PM, Bruno Cardoso
> Lopes<bruno.cardoso at gmail.com> wrote:
>> Hi Jeffrey,
>> On Wed, Jul 15, 2009 at 4:40 PM, Jeffrey
>> Yasskin<jyasskin at google.com> wrote:
>>> Here's the patch.
>>> On Wed, Jul 15, 2009 at 12:38 PM, <jyasskin at gmail.com> wrote:
>>>> Reviewers: ,
>>>> Please take a look. I'll attach the patch shortly. Bruno,
>>>> let me know how I can make this fit better with your long-term
>>>> plans for
>>>> full Dwarf output.
>> The modifications Regarding the MCE are ok for me, I'd do it in the
>> same way :)
>> What do you think about using the name processDebugLoc instead of
> I used beginDebugLoc because "process" didn't tell me anything about
> what the DebugLoc meant. But I don't mind changing it back and will do
> so unless someone else hollers.
> Any other comments, from anyone?
The patch looks reasonable to me.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits