[all-commits] [llvm/llvm-project] 0aa75f: [SLP] put verifyFunction call behind EXPENSIVE_CHECKS
RotateRight via All-commits
all-commits at lists.llvm.org
Sun Jan 10 10:05:35 PST 2021
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: 0aa75fb12faa04e07ba1a6e334605357b6a159c9
https://github.com/llvm/llvm-project/commit/0aa75fb12faa04e07ba1a6e334605357b6a159c9
Author: Sanjay Patel <spatel at rotateright.com>
Date: 2021-01-10 (Sun, 10 Jan 2021)
Changed paths:
M llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
Log Message:
-----------
[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
More information about the All-commits
mailing list