<p dir="ltr"><br>
On Mar 20, 2014 2:14 PM, "Adrian Prantl" <<a href="mailto:aprantl@apple.com">aprantl@apple.com</a>> wrote:<br>
><br>
> Hi Eric & Chandler,<br>
><br>
> it’s been a while, and I was just thinking about what we could do to make this move forward a little faster. I understand that the more time is passing the patch grows larger and large which makes it harder to review.</p>

<p dir="ltr">I'm confused as to why the patch would grow larger over time... But yes, prolonged review is expensive.</p>
<p dir="ltr">> We could, for instance split out the SROA changes into a separate patch, and then focus on the IR and DwarfDebug changes initially, with the type legalizer providing the necessary test cases.</p>
<p dir="ltr">If you can handcraft (or otherwise) it that is what you expect sroa to produce and demonstrate that CodeGen (DwarfDebug, etc) produces the right stuff - that seems good.</p>
<p dir="ltr">Assuming the ir format is easily agreed upon.</p>
<p dir="ltr">><br>
> cheers,<br>
> adrian<br>
><br>
><br>
> On Mar 18, 2014, at 5:55 PM, Adrian Prantl <<a href="mailto:aprantl@apple.com">aprantl@apple.com</a>> wrote:<br>
><br>
> >  Rebased on r204198.<br>
> >  Moved the DWARF expression output for DW_op_pieces out of emitDebugLocEntry::emitDebugLocEntry() and into a DWARFPieceEmitter. Should be more readable this way.<br>
> ><br>
> > Hi #llvm,<br>
> ><br>
> > <a href="http://llvm-reviews.chandlerc.com/D2680">http://llvm-reviews.chandlerc.com/D2680</a><br>
> ><br>
> > CHANGE SINCE LAST DIFF<br>
> >  <a href="http://llvm-reviews.chandlerc.com/D2680?vs=7650&id=7931#toc">http://llvm-reviews.chandlerc.com/D2680?vs=7650&id=7931#toc</a><br>
> ><br>
> > Files:<br>
> >  docs/SourceLevelDebugging.rst<br>
> >  include/llvm/CodeGen/AsmPrinter.h<br>
> >  include/llvm/CodeGen/MachineInstr.h<br>
> >  include/llvm/CodeGen/SelectionDAG.h<br>
> >  include/llvm/IR/DIBuilder.h<br>
> >  include/llvm/IR/DebugInfo.h<br>
> >  lib/CodeGen/AsmPrinter/AsmPrinterDwarf.cpp<br>
> >  lib/CodeGen/AsmPrinter/DwarfDebug.cpp<br>
> >  lib/CodeGen/AsmPrinter/DwarfDebug.h<br>
> >  lib/CodeGen/AsmPrinter/DwarfUnit.cpp<br>
> >  lib/CodeGen/SelectionDAG/InstrEmitter.cpp<br>
> >  lib/CodeGen/SelectionDAG/LegalizeTypes.cpp<br>
> >  lib/CodeGen/SelectionDAG/SelectionDAG.cpp<br>
> >  lib/IR/DIBuilder.cpp<br>
> >  lib/IR/DebugInfo.cpp<br>
> >  lib/Transforms/Scalar/SROA.cpp<br>
> >  test/DebugInfo/X86/dbg-large-unsigned-const.ll<br>
> >  test/DebugInfo/X86/pieces-1.ll<br>
> >  test/DebugInfo/X86/sdagsplit-1.ll<br>
> >  test/DebugInfo/X86/sdagsplit-2.ll<br>
> >  test/DebugInfo/X86/sroasplit-1.ll<br>
> >  test/DebugInfo/X86/sroasplit-2.ll<br>
> > <D2680.7.patch><br>
><br>
</p>