Removed unrelated change

Rename cppcoreguidelines-noexcept-move-constructor to cppcoreguidelines-noexcept-move-operations

Bumping to ask for reviews

Split the changes into 2 new checks and added cppcoreguidelines aliases for all of them.

If someone could be so kind as to push this on my behalf, it would be greatly appreciated

Implement suggested changes.

Well first of all thanks for the feedback. I agree not renaming a check which is already used and splitting each special case into it's own check seems like the easier and better way to go.

Minor code cleanup

Fix docs

Handle the function calling case

I've also notices that we don't handle this case

std::endl(std::cout);

Remove std::ends from the check

Implement suggested fixes

Rename to performance-avoid-endl

This check is mostly working. Added notes about problems I know about.

I'm fine with pushing this change, but since I don't have commit access someone would need to commit this for me

Now I would kindly as for someone to commit this on my behalf.

Remove unneccesary braces

In D147376#4239421, @PiotrZSL wrote:

LGTM.

In D147411#4241016, @PiotrZSL wrote:

LGTM, will you commit this or want someone else to commit this ?

Implement suggested way of checking for anonymous structs
Add a new more tests

Fixed docs and cleanup tests a bit

Implement review suggestions

Use llvm::DenseMap instead of std::map
Add some more test cases

Sprinkle some more const

In D146922#4227832, @PiotrZSL wrote:

Try refactoring code a lite bit, check suggestions, add tests for typedefs, and inheritance from template type.

Move functionality to a seperate file
Also the noexcept move constrcutor check now only runs with exceptions enabled like other checks.

Implemented suggested fixes from reviews

@PiotrZSL thanks for the quick response.