[PATCH] D146922: [clang-tidy] Fix false positve for defaulted move constructor in performance-noexcept-move-constructor

André Schackier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 27 05:07:58 PDT 2023


AMS21 added a comment.

@PiotrZSL thanks for the quick response.

I've tried to work on the `Field` test case but seem to have hit a roadblock. From what I can see the `FunctionProtoType` exception specificaion is `EST_Unevaluated` which causes it to give up because of the check on Line 41.
I'm not very familiar with the codebase but looked around and found the `ResolveExceptionSpec()` function inside `Sema.h` which seems to do exactly what I need: tell me if the implicit definition is throwing or not.
But I can't quite seem to figure out how to get a `Sema` object or construct one to use said function.
Or maybe there is another more clever way to go about it which I'm not aware of?

Any pointers into the right direction would be kindly appreciated.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146922



More information about the cfe-commits mailing list