[PATCH] D130282: [llvm][FileCheck] Fix unit tests failures with EXPENSIVE_CHECKS
Kazu Hirata via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 21 10:17:50 PDT 2022
kazu added a comment.
I am inclined to having the check unconditionally to be more robust. It is super cheap. The condition always evaluates to false without `EXPENSIVE_CHECKS`, so it gets along with the branch predictor. Plus, it would be nice not to rely on the fact that `EXPENSIVE_CHECKS` enables `_GLIBCXX_DEBUG`. Thoughts?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130282/new/
https://reviews.llvm.org/D130282
More information about the llvm-commits
mailing list