andreybokhanko marked an inline comment as done.
================
Comment at: test/SemaCXX/MicrosoftExtensions.cpp:89
@@ +88,3 @@
+void foo_unaligned(int *arg) {}
+void foo_unaligned(__unaligned int *arg) {}
+void foo_unaligned(int arg) {} // expected-note {{previous definition is here}}
----------------
Reid, thanks for looking into this patch!
I added more comprehensive overloading tests.
As for printing a warning, I added it -- in C mode. In C++ mode we don't allow
types conversion that loses qualifiers. I do the same for __unaligned. If you
want me to implement printing a warning in C++ mode as well (as MS compiler
does), I would appreciate any hints on where the best place to do it. Putting
it directly into code that verifies qualifiers compatibility seems to be odd.
Andrey
http://reviews.llvm.org/D18596
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
