[PATCH] D23518: Split DescribeAddressIf* (and DescribeHeapAddress) between a function that gets all the information (structured), and one that prints it.

Filipe Cabecinhas via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 15 10:31:56 PDT 2016


Bah, Phabricator doesn't show the commits one by one. I will do what I have
in the commits where I split up the shadow stuff, then move some code, then
split up the rest one by one. Will post reviews separately soon.

Thank you,

 Filipe

On Monday, 15 August 2016, Kostya Serebryany <kcc at google.com> wrote:

> kcc added a comment.
>
> This is a large change and will take quadratic time for review.
> Please help me speed it up by splitting the change into several.
> Especially, separate simple refactoring (moving code around that does not
> change functionality) from actual changes in the code (even renaming
> functions).
>
>
> https://reviews.llvm.org/D23518
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160815/a64077e5/attachment.html>


More information about the llvm-commits mailing list