On Thu, Jul 14, 2011 at 4:10 PM, Andrew Stubbs <a...@codesourcery.com> wrote: > On 12/07/11 12:04, Richard Guenther wrote: >> >> I wonder if we want to restrict the WIDEN_* operations to operate >> on types that have matching type/mode precision(**). > > I've now modified the patch to allow bitfields, or other case where the > precision is smaller than the mode-size. I've also addressed the formatting > issues you pointed out (and in fact reorganised the code slightly to make > the rest of the series a bit cleaner). > > As in the previous version of this patch, it's necessary to convert the > input values to the proper mode for the machine instruction, so the basic > tools for supporting the bitfields were already there - I just had to tweak > the conditionals to take bitfields into account. > > The only this I haven't done is modified tree.def. Looking at it though, I > don't thing any needs changing? The code is still valid, and the comments > are correct (in fact, they may have been wrong before).
Ah, it indeed talks about at least twice the precision already. > Is this version OK? Ok. Thanks, Richard. > Andrew >
