[PATCH] D122181: Cleanup includes: Transforms/Instrumentation & Transforms/Vectorize

[serge via Phabricator via llvm-commits]

serge-sans-paille added inline comments.


================
Comment at: llvm/lib/Transforms/Vectorize/SLPVectorizer.cpp:74
 #include "llvm/IR/ValueHandle.h"
+#ifdef EXPENSIVE_CHECKS
 #include "llvm/IR/Verifier.h"
----------------
MaskRay wrote:
> Consider including it unconditionally. The header is small
Unfortunately, this doesn't help with IWYU reports (it leads to a false unused incldue when running the checker under `-UEXPENSIVE_CHECK`), that's why I'm guarding them.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122181