erichkeane added a comment.

In D126907#3852909 <https://reviews.llvm.org/D126907#3852909>, @BertalanD wrote:

> I tried out D135772 <https://reviews.llvm.org/D135772>, and our build got 
> significantly farther than before! I unfortunately discovered another piece 
> of code that pre babdef27c503c0bbbcc017e9f88affddda90ea4e 
> <https://reviews.llvm.org/rGbabdef27c503c0bbbcc017e9f88affddda90ea4e> Clang 
> and GCC accept in C++20 mode, but Clang trunk does not 
> (https://godbolt.org/z/q1q4nfobK):
>
>   struct String {
>     String(char *);
>     bool operator==(String const &);
>     void operator!=(String const &);
>   };
>   
>   extern char* c;
>   extern String s;
>   int test() {
>     return c == s;
>   }
>
>
>
>   <source>:10:12: error: invalid operands to binary expression ('char *' and 
> 'String')
>     return c == s;
>            ~ ^  ~

Thanks!  I'll look into that as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126907

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

Reply via email to