Re: Patch: Some potential warnings for C++ bootstrap

On 11/14/14 13:41, Kai Tietz wrote: Hello, this patch fixes some potential warnings for C++ bootstrap. I noticed them while working on the delayed folding for C++-FE on boostrap. ChangeLog 2014-11-14 Kai Tietz kti...@redhat.com * dwarf2out.c(output_loc_operands): Make sure

Patch: Some potential warnings for C++ bootstrap

Hello, this patch fixes some potential warnings for C++ bootstrap. I noticed them while working on the delayed folding for C++-FE on boostrap. ChangeLog 2014-11-14 Kai Tietz kti...@redhat.com * dwarf2out.c(output_loc_operands): Make sure that comparison is done on same sign

Re: Patch: Some potential warnings for C++ bootstrap

On 11/14/2014 03:41 PM, Kai Tietz wrote: - gcc_assert (val1-v.val_unsigned == DWARF2_ADDR_SIZE); + gcc_assert (val1-v.val_unsigned + == (unsigned HOST_WIDE_INT) DWARF2_ADDR_SIZE); These changes should not be necessary; the warning should check the folded versions of the

Re: Patch: Some potential warnings for C++ bootstrap

2014-11-14 21:45 GMT+01:00 Jason Merrill ja...@redhat.com: On 11/14/2014 03:41 PM, Kai Tietz wrote: - gcc_assert (val1-v.val_unsigned == DWARF2_ADDR_SIZE); + gcc_assert (val1-v.val_unsigned + == (unsigned HOST_WIDE_INT) DWARF2_ADDR_SIZE); These changes should not be