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
