[PATCH] D94576: [LoopVectorize] Guard verifyFunction with EXPENSIVE_CHECKS macro

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 13 08:55:43 PST 2021


lebedev.ri added a comment.

In D94576#2495791 <https://reviews.llvm.org/D94576#2495791>, @lebedev.ri wrote:

> In D94576#2495787 <https://reviews.llvm.org/D94576#2495787>, @tislam wrote:
>
>> In D94576#2495786 <https://reviews.llvm.org/D94576#2495786>, @lebedev.ri wrote:
>>
>>> So are the compilation time issues reproducible with vanilla llvm, or only in your local branch?
>>> If latter, shouldn't the fix (which isn't hiding the verification under) belong to the branch?
>>
>> Hi @lebedev.ri, the first profile is with the vanilla llvm.
>
> That is what the message said yes.

@tislam to make progress here, can you share a reproducer that shows the problematic compile time implications on vanilla llvm?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94576/new/

https://reviews.llvm.org/D94576



More information about the llvm-commits mailing list