This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: [PATCH] Fix for PR c/57563


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
joseph@codesourcery.com


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]