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. <br><br><div class="gmail_quote"><div dir="ltr">On Fri, Nov 30, 2018, 8:25 AM Phabricator via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">This revision was not accepted when it landed; it landed in state "Needs Review".<br>
This revision was automatically updated to reflect the committed changes.<br>
Closed by commit rL347999: [BTF] Add BTF DebugInfo (authored by yhs, committed by ).<br>
<br>
Changed prior to commit:<br>
  <a href="https://reviews.llvm.org/D53736?vs=174527&id=176133#toc" rel="noreferrer" target="_blank">https://reviews.llvm.org/D53736?vs=174527&id=176133#toc</a><br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D53736/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D53736/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D53736" rel="noreferrer" target="_blank">https://reviews.llvm.org/D53736</a><br>
<br>
Files:<br>
  llvm/trunk/include/llvm/BinaryFormat/BTF.def<br>
  llvm/trunk/include/llvm/BinaryFormat/BTF.h<br>
  llvm/trunk/include/llvm/MC/MCObjectFileInfo.h<br>
  llvm/trunk/lib/CodeGen/AsmPrinter/AsmPrinter.cpp<br>
  llvm/trunk/lib/CodeGen/AsmPrinter/BTFDebug.cpp<br>
  llvm/trunk/lib/CodeGen/AsmPrinter/BTFDebug.h<br>
  llvm/trunk/lib/CodeGen/AsmPrinter/BTFDebugStub.cpp<br>
  llvm/trunk/lib/CodeGen/AsmPrinter/CMakeLists.txt<br>
  llvm/trunk/lib/CodeGen/AsmPrinter/DebugHandlerBase.cpp<br>
  llvm/trunk/lib/CodeGen/AsmPrinter/DebugHandlerBase.h<br>
  llvm/trunk/lib/MC/MCObjectFileInfo.cpp<br>
  llvm/trunk/test/DebugInfo/BTF/array-1d-char.ll<br>
  llvm/trunk/test/DebugInfo/BTF/array-1d-int.ll<br>
  llvm/trunk/test/DebugInfo/BTF/array-2d-int.ll<br>
  llvm/trunk/test/DebugInfo/BTF/array-size-0.ll<br>
  llvm/trunk/test/DebugInfo/BTF/array-typedef.ll<br>
  llvm/trunk/test/DebugInfo/BTF/binary-format.ll<br>
  llvm/trunk/test/DebugInfo/BTF/char.ll<br>
  llvm/trunk/test/DebugInfo/BTF/enum-basic.ll<br>
  llvm/trunk/test/DebugInfo/BTF/func-func-ptr.ll<br>
  llvm/trunk/test/DebugInfo/BTF/func-non-void.ll<br>
  llvm/trunk/test/DebugInfo/BTF/func-source.ll<br>
  llvm/trunk/test/DebugInfo/BTF/func-typedef.ll<br>
  llvm/trunk/test/DebugInfo/BTF/func-unused-arg.ll<br>
  llvm/trunk/test/DebugInfo/BTF/func-void.ll<br>
  llvm/trunk/test/DebugInfo/BTF/fwd-no-define.ll<br>
  llvm/trunk/test/DebugInfo/BTF/fwd-with-define.ll<br>
  llvm/trunk/test/DebugInfo/BTF/int.ll<br>
  llvm/trunk/test/DebugInfo/BTF/lit.local.cfg<br>
  llvm/trunk/test/DebugInfo/BTF/longlong.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ptr-const-void.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ptr-func-1.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ptr-func-2.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ptr-func-3.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ptr-int.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ptr-void.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ptr-volatile-const-void.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ptr-volatile-void.ll<br>
  llvm/trunk/test/DebugInfo/BTF/restrict-ptr.ll<br>
  llvm/trunk/test/DebugInfo/BTF/short.ll<br>
  llvm/trunk/test/DebugInfo/BTF/struct-anon.ll<br>
  llvm/trunk/test/DebugInfo/BTF/struct-basic.ll<br>
  llvm/trunk/test/DebugInfo/BTF/struct-bitfield-typedef.ll<br>
  llvm/trunk/test/DebugInfo/BTF/struct-enum.ll<br>
  llvm/trunk/test/DebugInfo/BTF/uchar.ll<br>
  llvm/trunk/test/DebugInfo/BTF/uint.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ulonglong.ll<br>
  llvm/trunk/test/DebugInfo/BTF/union-array-typedef.ll<br>
  llvm/trunk/test/DebugInfo/BTF/ushort.ll<br>
<br>
</blockquote></div>