[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