mstorsjo added a comment.

In D67856#1677356 <https://reviews.llvm.org/D67856#1677356>, @davide wrote:

> If this doesn't break clang and gcc, fine.


FWIW, it was with clang that I ran into it, haven't tested building lldb with 
gcc.

But the issue can be tested with a snippet like this:

  class Foo {
  public: 
          Foo::Foo() {}
  };

With MSVC, this passes fine without any warnings. With clang-cl (or plain clang 
with an msvc target), it builds but warns (`-Wmicrosoft-extra-qualification`), 
with gcc it's an error (that can be waived with `-fpermissive`), and with clang 
a non-msvc target, it's an unrecoverable error.


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D67856



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

Reply via email to