[SLP] put verifyFunction call behind EXPENSIVE_CHECKS
authorSanjay Patel <spatel@rotateright.com>
Sun, 10 Jan 2021 17:31:05 +0000 (12:31 -0500)
committerSanjay Patel <spatel@rotateright.com>
Sun, 10 Jan 2021 17:32:21 +0000 (12:32 -0500)
commit0aa75fb12faa04e07ba1a6e334605357b6a159c9
treeda00fd665a044ebc58ef32e4431420d62cbdce63
parente3d3dbd33930b7bc879b522d65f6d27c1677253e
[SLP] put verifyFunction call behind EXPENSIVE_CHECKS

A severe compile-time slowdown from this call is noted in:
https://llvm.org/PR48689
My naive fix was to put it under LLVM_DEBUG ( 267ff79 ),
but that's not limiting in the way we want.
This is a quick fix (or we could just remove the call completely
and rely on some later pass to discover potentially wrong IR?).
A bigger/better fix would be to improve/limit verifyFunction()
as noted in:
https://llvm.org/PR47712

Differential Revision: https://reviews.llvm.org/D94328
llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp