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 PR15791



On Jan 27, 2005, at 10:25 AM, Richard Guenther wrote:


On Thu, 27 Jan 2005, Jakub Jelinek wrote:

Don't both arguments of the compare need to have the
same type?  With the above code, you might very well
use (int) for one of the operand and likely size_t
for the other one.

I honestly don't know - doesn't build2 handle this? Also this would question the validity of the optimization, no? The simplest way to fix this would probably be to check for type equality of the offsets.

No build2 never has handled type mismatch. (this is unlike cp_build_binary_op in the C++ front-end).


-- pinski



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