<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Jul 21, 2015, at 12:39 PM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="">dblaikie@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><br class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Tue, Jul 21, 2015 at 12:31 PM, Pete Cooper <span dir="ltr" class=""><<a href="mailto:peter_cooper@apple.com" target="_blank" class="">peter_cooper@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Do you think this happens enough that it should be issued by clang with -Wcompat?<br class=""></blockquote><div class=""><br class=""></div><div class="">I haven't really looked at what else we put under that flag, if anything - so I'm not sure if this would fit/meet the bar.<br class=""></div></div></div></div></div></blockquote>I haven’t looked either. Just wanted to suggest it in case it did fit.<br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""><br class="">I'm personally not too fussed by it. I just consider non-clang builds of clang to be things we need to maintain, but not things we need to worry about too much. So I'm usually OK with breaking GCC and MSVC buildbots and fixing forwards for these little incompatibilities.<br class=""></div></div></div></div></div></blockquote>Yeah, actually i’m fine with that too. Hopefully the bots don’t break so often that this is major blocker for people. If it is we can always revisit -Wcompat or other solutions later.</div><div><br class=""></div><div>Cheers,</div><div>Pete<br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""><br class="">- Dave</div><div class=""> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="">
Cheers,<br class="">
Pete<br class="">
<div class="HOEnZb"><div class="h5">> On Jul 21, 2015, at 11:59 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" class="">dblaikie@gmail.com</a>> wrote:<br class="">
><br class="">
> Author: dblaikie<br class="">
> Date: Tue Jul 21 13:59:10 2015<br class="">
> New Revision: 242826<br class="">
><br class="">
> URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject-3Frev-3D242826-26view-3Drev&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=9DVGEZsDEakYgf8IvAQq0CjnD-LfutjvJJKefE5hXXg&s=Cg_RbQK9pTdhyOfhgiiOC2IM26-Cx-FPBucfWIuvfLk&e=" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project?rev=242826&view=rev</a><br class="">
> Log:<br class="">
> Fix GCC build due to shadowing<br class="">
><br class="">
> Modified:<br class="">
> cfe/trunk/lib/CodeGen/CodeGenFunction.h<br class="">
><br class="">
> Modified: cfe/trunk/lib/CodeGen/CodeGenFunction.h<br class="">
> URL: <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__llvm.org_viewvc_llvm-2Dproject_cfe_trunk_lib_CodeGen_CodeGenFunction.h-3Frev-3D242826-26r1-3D242825-26r2-3D242826-26view-3Ddiff&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=BSqEv9KvKMW_Ob8SyngJ70KdZISM_ASROnREeq0cCxk&m=9DVGEZsDEakYgf8IvAQq0CjnD-LfutjvJJKefE5hXXg&s=PxNF0vToYIwC4TuzPE3iTh5jc_ZVWEkKA6hejQeLDk8&e=" rel="noreferrer" target="_blank" class="">http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CodeGenFunction.h?rev=242826&r1=242825&r2=242826&view=diff</a><br class="">
> ==============================================================================<br class="">
> --- cfe/trunk/lib/CodeGen/CodeGenFunction.h (original)<br class="">
> +++ cfe/trunk/lib/CodeGen/CodeGenFunction.h Tue Jul 21 13:59:10 2015<br class="">
> @@ -3006,8 +3006,8 @@ public:<br class="">
> "Extra arguments in non-variadic function!");<br class="">
><br class="">
> // If we still have any arguments, emit them using the type of the argument.<br class="">
> - for (auto *Arg : llvm::make_range(Arg, ArgRange.end()))<br class="">
> - ArgTypes.push_back(getVarArgType(Arg));<br class="">
> + for (auto *A : llvm::make_range(Arg, ArgRange.end()))<br class="">
> + ArgTypes.push_back(getVarArgType(A));<br class="">
><br class="">
> EmitCallArgs(Args, ArgTypes, ArgRange, CalleeDecl, ParamsToSkip);<br class="">
> }<br class="">
><br class="">
><br class="">
> _______________________________________________<br class="">
> cfe-commits mailing list<br class="">
> <a href="mailto:cfe-commits@cs.uiuc.edu" class="">cfe-commits@cs.uiuc.edu</a><br class="">
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank" class="">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br class="">
<br class="">
</div></div></blockquote></div><br class=""></div></div>
</div></blockquote></div><br class=""></body></html>