[PATCH] [MSVC] Improved __noop support (https://llvm.org/bugs/show_bug.cgi?id=14081)

Charlie Turner charlie.turner at arm.com
Tue Apr 14 15:49:23 PDT 2015


================
Comment at: test/Sema/builtin-ms-noop-errors.cpp:16
@@ +15,3 @@
+  int var1, var2, var3, var4, var5, var6, var7, var8;
+  ((__noop))(function2<void>()); // expected-note{{'__noop' is a builtin with type 'int ()'}}
+  __noop(Z(), Z(), (function2<int>()), Z1(function2<int>()));
----------------
I find checking this note weird. I know `__noop` is defined as taking `...` from the Builtins.def file, but `ASTContext::GetBuiltinType` decides to ignore it and generate the empty argument list instead.

http://reviews.llvm.org/D9000

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the cfe-commits mailing list