[Parallel_libs-commits] [PATCH] D24468: [SE] Add .clang-format

[Justin Lebar via Parallel_libs-commits]

jlebar added a comment.

In https://reviews.llvm.org/D24468#541585, @jhen wrote:

> I added the noexcept move constructor check back in because I learned how to get rid of false positives. It turns out that the current check won't allow you to explicitly default a noexcept constructor at the same time as you declare it. To get around this, I just moved the explicit default definitions out-of-line. Now we can keep this useful check.


That...sort of makes sense.  There is a subtle difference between `=default` inside the class definition, which I think means "whatever the compiler will generate by default, including not generating an implementation if it can't," and putting the `=default` outside the class definition, which means "provide me the default implementation of this function."


https://reviews.llvm.org/D24468