On Mon, 10 Jun 2013, Iyer, Balaji V wrote: > > This version is better, but if removing an EXCESS_PRECISION_EXPR there > > caused > > problems, why is it OK to remove CONVERT_EXPR and NOP_EXPR like you still > > do - won't that also cause type mismatches (at least if the conversions are > > to > > types that count as sufficiently different for GIMPLE purposes - say > > conversions > > between 32-bit and 64-bit integers)? Maybe you actually need to fold > > without > > removing any such wrappers first at all? > > I looked into it and they were an artifact of previous implementation. > Those while loops were not even being entered. Thus, I took them out. > Here is a fixed patch.
Thanks, this patch is OK. -- Joseph S. Myers jos...@codesourcery.com