curdeius added a comment.

With your patch I still see the bug in this case:

  struct S {
    constexpr S(const S &) = default;
    void f()               = default;
    S &operator/**/        =(S) {}
  };

Mind the `operator  =`.
It happens every time that `operator=` has only declaration (no definition 
body).

Another failure:

  struct S {
    /*comment*/ S(const S &) = default;
    void f()                 = default;
    S &operator/**/          =(S) {}
  };

As mentioned yesterday, comment between `operator` and `=`.

Note, `/*comment*/` is there just to make the spacing more visible. It's 
unrelated to the bug otherwise.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D117421

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to