Review Ping (was Re: [PATCH] D53736: [BTF] Add BTF DebugInfo)
Yonghong Song via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 13 15:55:32 PST 2018
Hi, Eric,
This is Yonghong, pinging again. Christmas is fast coming.
I am not sure whether you got any time to review the patch or not
as I did not see any Phabricator update yet...
Thanks!
Yonghong
On 12/7/18 11:11 AM, Eric Christopher wrote:
> Hi!
>
> I'm here, and as you can tell have been working my way through my review
> queue. I'm trying to get to them mostly in the order I received them
> originally so I should get to it soon.
>
> Thanks!
>
> -eric
>
> On Fri, Dec 7, 2018 at 9:06 AM Yonghong Song <yhs at fb.com
> <mailto:yhs at fb.com>> wrote:
>
> Hi, Eric,
>
> I noticed that the llvm 8.0.0 will be branched out on Jan 16, 2019.
> I am not sure about your holiday schedule and I worried that this may
> get even more delayed as Christmas approaches. It will be really good
> if you can allocate some time to review this patch soon.
>
> Also, I noticed that my multiple reminders on phabricator generated
> no response from you and I am not even sure that this email will
> be able to reach you or not. Previous email communication is due
> to merge and revert the patch. Further back an email communication
> is an email directly addressed to you.
>
> If you see this email and really do not have time to review it now,
> could you still reply like you saw this email and will review later
> so I know that you can still reach/ping you through this email thread?
>
> Thanks,
>
> Yonghong
>
> On 12/4/18 12:42 PM, Yonghong Song wrote:
> >
> > Hi, Eric,
> >
> > Just a kind reminder that it would be great if you can squeeze
> some time
> > to review the patch.
> >
> > Thanks!
> >
> > Yonghong
> >
> >
> > On 11/30/18 8:56 AM, Eric Christopher wrote:
> >> FWIW I do like the new patch direction and hope to get to this
> shortly.
> >>
> >> On Fri, Nov 30, 2018, 8:51 AM Yonghong Song <yhs at fb.com
> <mailto:yhs at fb.com>
> >> <mailto:yhs at fb.com <mailto:yhs at fb.com>>> wrote:
> >>
> >>
> >> I will revert it now.
> >>
> >> On 11/30/18 8:49 AM, Eric Christopher wrote:
> >> > So Adrian didn't accept this and it touches code that Alexei
> >> doesn't
> >> > have any real experience (as far as I know) with so it isn't
> >> appropriate
> >> > for you to land this without approval. I understand
> you've been
> >> waiting
> >> > for a while, but that's not a good enough reason. Please
> revert
> >> for now
> >> > until we can look.
> >> >
> >> > On Fri, Nov 30, 2018, 8:25 AM Phabricator via Phabricator
> >> > <reviews at reviews.llvm.org
> <mailto:reviews at reviews.llvm.org> <mailto:reviews at reviews.llvm.org
> <mailto:reviews at reviews.llvm.org>>
> >> <mailto:reviews at reviews.llvm.org
> <mailto:reviews at reviews.llvm.org> <mailto:reviews at reviews.llvm.org
> <mailto:reviews at reviews.llvm.org>>>>
> >> wrote:
> >> >
> >> > This revision was not accepted when it landed; it
> landed in
> >> state
> >> > "Needs Review".
> >> > This revision was automatically updated to reflect the
> >> committed
> >> > changes.
> >> > Closed by commit rL347999: [BTF] Add BTF DebugInfo
> (authored
> >> by yhs,
> >> > committed by ).
> >> >
> >> > Changed prior to commit:
> >> > https://reviews.llvm.org/D53736?vs=174527&id=176133#toc
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736-3Fvs-3D174527-26id-3D176133-23toc&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=3C_IPMfp3N4i6wAg7_gPPzqF6NXABR9GoDplHh0UGZY&s=nv2LwUrgNR9SpNMGppNdl6k-liRFdpHa2waRzC3Ou0Q&e=>
> >>
> >>
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736-3Fvs-3D174527-26id-3D176133-23toc&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=eV5zffARXFpMPlLta8ho2Kxy9PdTMzUkxV9bVTtGKuI&s=ffbr3Y43MuGLV5hak5U2PehlCma1LfFogd9MuKBgSrk&e=>
>
> >>
> >> >
> >>
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736-3Fvs-3D174527-26id-3D176133-23toc&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=oest39Lss5_5lx6NZgWa9qhPYwGcCe2RjWValScV-rw&s=zPdsdNZ1_DF0UBWsU_9yoevmReH_8IqNEs335p6L0gs&e=>
> >>
> >> >
> >> > Repository:
> >> > rL LLVM
> >> >
> >> > CHANGES SINCE LAST ACTION
> >> > https://reviews.llvm.org/D53736/new/
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736_new_&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=3C_IPMfp3N4i6wAg7_gPPzqF6NXABR9GoDplHh0UGZY&s=NXoXV8rwJtyHZtxHe4hVYJdsvjG0hzIYty6QMAHKosU&e=>
> >>
> >>
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736_new_&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=eV5zffARXFpMPlLta8ho2Kxy9PdTMzUkxV9bVTtGKuI&s=yrmL8h7o4HndoVuzMkR1VcwF2SBKExGZDBXUwg6Gm_Y&e=>
>
> >>
> >> >
> >>
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736_new_&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=oest39Lss5_5lx6NZgWa9qhPYwGcCe2RjWValScV-rw&s=OsDyTNnwqfvfxsEmM2PHsDO9fRXyB3fe9fCb2UmxtsY&e=>
> >>
> >> >
> >> > https://reviews.llvm.org/D53736
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=3C_IPMfp3N4i6wAg7_gPPzqF6NXABR9GoDplHh0UGZY&s=eyc-jx24kndA6cNRVVPDA86tHaF9uXtoiRk2qnDoHL4&e=>
> >>
> >>
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=eV5zffARXFpMPlLta8ho2Kxy9PdTMzUkxV9bVTtGKuI&s=Q3ec5ZHnBt6_mtJh4cqHwIT1ETLlMvEFzJn4HI_ncYU&e=>
>
> >>
> >> >
> >>
> <https://urldefense.proofpoint.com/v2/url?u=https-3A__reviews.llvm.org_D53736&d=DwMFaQ&c=5VD0RTtNlTh3ycd41b3MUw&r=DA8e1B5r073vIqRrFz7MRA&m=oest39Lss5_5lx6NZgWa9qhPYwGcCe2RjWValScV-rw&s=YSLbTE6YZuqlk1kytiaV1RA80fKpF4HJU11eMc1gA2Q&e=>
> >>
> >> >
> >> > Files:
> >> > llvm/trunk/include/llvm/BinaryFormat/BTF.def
> >> > llvm/trunk/include/llvm/BinaryFormat/BTF.h
> >> > llvm/trunk/include/llvm/MC/MCObjectFileInfo.h
> >> > llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
> >> > llvm/trunk/lib/CodeGen/AsmPrinter/BTFDebug.cpp
> >> > llvm/trunk/lib/CodeGen/AsmPrinter/BTFDebug.h
> >> > llvm/trunk/lib/CodeGen/AsmPrinter/BTFDebugStub.cpp
> >> > llvm/trunk/lib/CodeGen/AsmPrinter/CMakeLists.txt
> >> > llvm/trunk/lib/CodeGen/AsmPrinter/DebugHandlerBase.cpp
> >> > llvm/trunk/lib/CodeGen/AsmPrinter/DebugHandlerBase.h
> >> > llvm/trunk/lib/MC/MCObjectFileInfo.cpp
> >> > llvm/trunk/test/DebugInfo/BTF/array-1d-char.ll
> >> > llvm/trunk/test/DebugInfo/BTF/array-1d-int.ll
> >> > llvm/trunk/test/DebugInfo/BTF/array-2d-int.ll
> >> > llvm/trunk/test/DebugInfo/BTF/array-size-0.ll
> >> > llvm/trunk/test/DebugInfo/BTF/array-typedef.ll
> >> > llvm/trunk/test/DebugInfo/BTF/binary-format.ll
> >> > llvm/trunk/test/DebugInfo/BTF/char.ll
> >> > llvm/trunk/test/DebugInfo/BTF/enum-basic.ll
> >> > llvm/trunk/test/DebugInfo/BTF/func-func-ptr.ll
> >> > llvm/trunk/test/DebugInfo/BTF/func-non-void.ll
> >> > llvm/trunk/test/DebugInfo/BTF/func-source.ll
> >> > llvm/trunk/test/DebugInfo/BTF/func-typedef.ll
> >> > llvm/trunk/test/DebugInfo/BTF/func-unused-arg.ll
> >> > llvm/trunk/test/DebugInfo/BTF/func-void.ll
> >> > llvm/trunk/test/DebugInfo/BTF/fwd-no-define.ll
> >> > llvm/trunk/test/DebugInfo/BTF/fwd-with-define.ll
> >> > llvm/trunk/test/DebugInfo/BTF/int.ll
> >> > llvm/trunk/test/DebugInfo/BTF/lit.local.cfg
> >> > llvm/trunk/test/DebugInfo/BTF/longlong.ll
> >> > llvm/trunk/test/DebugInfo/BTF/ptr-const-void.ll
> >> > llvm/trunk/test/DebugInfo/BTF/ptr-func-1.ll
> >> > llvm/trunk/test/DebugInfo/BTF/ptr-func-2.ll
> >> > llvm/trunk/test/DebugInfo/BTF/ptr-func-3.ll
> >> > llvm/trunk/test/DebugInfo/BTF/ptr-int.ll
> >> > llvm/trunk/test/DebugInfo/BTF/ptr-void.ll
> >> >
> llvm/trunk/test/DebugInfo/BTF/ptr-volatile-const-void.ll
> >> > llvm/trunk/test/DebugInfo/BTF/ptr-volatile-void.ll
> >> > llvm/trunk/test/DebugInfo/BTF/restrict-ptr.ll
> >> > llvm/trunk/test/DebugInfo/BTF/short.ll
> >> > llvm/trunk/test/DebugInfo/BTF/struct-anon.ll
> >> > llvm/trunk/test/DebugInfo/BTF/struct-basic.ll
> >> >
> llvm/trunk/test/DebugInfo/BTF/struct-bitfield-typedef.ll
> >> > llvm/trunk/test/DebugInfo/BTF/struct-enum.ll
> >> > llvm/trunk/test/DebugInfo/BTF/uchar.ll
> >> > llvm/trunk/test/DebugInfo/BTF/uint.ll
> >> > llvm/trunk/test/DebugInfo/BTF/ulonglong.ll
> >> > llvm/trunk/test/DebugInfo/BTF/union-array-typedef.ll
> >> > llvm/trunk/test/DebugInfo/BTF/ushort.ll
> >> >
> >>
>
More information about the llvm-commits
mailing list