[PATCH] D36311: [ThinLTO] Add GraphTraits for FunctionSummaries

Charles Saternos via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 13 08:44:13 PDT 2017


ncharlie marked 4 inline comments as done.
ncharlie added a comment.

In https://reviews.llvm.org/D36311#896973, @tejohnson wrote:

> In https://reviews.llvm.org/D36311#896969, @ncharlie wrote:
>
> > Any updates on this @tejohnson @davide ?
>
>
> Sorry - didn't realize this was ready for re-review! I had a question about the tests earlier (see below). The comment wasn't marked as done. Although I see now that the test has changed a bit - did you resolve this? I will take a look this morning.


Ah sorry - forgot to mark it. Yeah - it's corrected now. Updated the tests accordingly.


https://reviews.llvm.org/D36311





More information about the llvm-commits mailing list