[PATCH] D52825: [llvm-exegesis] Resolve variant classes in analysis.

[Clément Courbet via llvm-commits]

Thanks Fangrui, will do, sorry about the breakage.

On Thu, Oct 4, 2018 at 5:36 AM Fangrui Song via Phabricator <
reviews at reviews.llvm.org> wrote:

> MaskRay added a comment.
>
> A Release build (where `#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)`
> is false) does not call `writeEscaped` and there will be no
> `SBWriteZeroLatency` in the output of
> `test/tools/llvm-exegesis/X86/analysis-uops-variant.test`
>
> I have removed the second check line to unbreak the test. Please fix it
> properly:)
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D52825
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181004/bd6a44ce/attachment.html>