[PATCH] D26967: Put opt-viewer critical items in parallel

Brian Cain via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 7 18:20:54 PST 2016


bcain added a comment.

In https://reviews.llvm.org/D26967#616015, @anemet wrote:

> Brian, just to make sure we're not deadlocked here waiting for each other.  I am still looking for an answer to this question:
>
> > I see consistent speed-ups but the resulting HTML directories don't match the original. It *seems* there are some extra remarks so I am assuming the uniquing does not work? Can you please look into it? After that I think this is ready to go in.


I'd hoped that might go away when I fixed the args getting passed to generate_report().

I'll run some tests to see if I can reproduce the problem.


Repository:
  rL LLVM

https://reviews.llvm.org/D26967





More information about the llvm-commits mailing list