[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