Thanks! Commit in r252104. ~Aaron
On Wed, Nov 4, 2015 at 7:05 PM, Richard Smith <rich...@metafoo.co.uk> wrote: > LGTM, thanks! > > On Wed, Nov 4, 2015 at 4:00 PM, Aaron Ballman via cfe-commits > <cfe-commits@lists.llvm.org> wrote: >> >> On Wed, Nov 4, 2015 at 5:46 PM, Richard Smith <rich...@metafoo.co.uk> >> wrote: >> > On Wed, Nov 4, 2015 at 2:45 PM, Richard Smith <rich...@metafoo.co.uk> >> > wrote: >> >> >> >> It'd be simpler and would more directly match the C specification (and >> >> would handle a few other cases better, such as placeholder types and >> >> atomic >> >> types) if you instead passed the operand through >> >> DefaultLvalueConversion >> > >> > >> > or rather, DefaultFunctionArrayLvalueConversion =) >> >> That was the magic incantation I didn't find before; thank you for >> pointing it out! Updated patch attached. >> >> ~Aaron >> >> _______________________________________________ >> cfe-commits mailing list >> cfe-commits@lists.llvm.org >> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits >> > _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
