[PATCH] D116085: [clang-tidy] Performance improvements for NOLINTBEGIN/END blocks
Nemanja Ivanovic via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 26 04:45:27 PST 2022
nemanjai added a comment.
In D116085#3272200 <https://reviews.llvm.org/D116085#3272200>, @nemanjai wrote:
> This is causing buildbot failures with `-Werror` (i.e. https://lab.llvm.org/buildbot/#/builders/57/builds/14322/steps/5/logs/stdio). Please fix or revert. The failure is:
>
> .../llvm-project/clang-tools-extra/clang-tidy/NoLintDirectiveHandler.cpp:204:38: error: moving a temporary object prevents copy elision [-Werror,-Wpessimizing-move]
> CompletedBlocks.emplace_back(std::move(Stack.pop_back_val()), NoLint);
Sorry, I didn't realize you already reverted as there was no mention in the revert of the original commit.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116085/new/
https://reviews.llvm.org/D116085
More information about the cfe-commits
mailing list