[PATCH-Android] [android/external/elfcopy] Modified to eliminate build errors using GCC 4.5/4.6.
asac at linaro.org
Wed Mar 16 12:35:19 UTC 2011
Ack, we discussed this today in android meeting and will take your
patches to a special toolchain branch that allows us to experiement
with 4.5/4.6 toolchain while not hiding the underlying problems.
Thanks for your work!
On Wed, Mar 16, 2011 at 1:01 PM, Sachin Kamat <sachin.kamat at linaro.org> wrote:
> Hi Arnd, Alexander,
> Some of the work arounds are just make shift solutions so that those
> dependent on the build can carry on with their work.
> Meanwhile as you guys said, the underlying problem needs to be addressed and
> fixed. This might take a while.
> On 16 March 2011 17:17, Arnd Bergmann <arnd at arndb.de> wrote:
>> On Wednesday 16 March 2011, Sachin Kamat wrote:
>> > --- a/elfcopy.c
>> > +++ b/elfcopy.c
>> > @@ -2456,7 +2456,6 @@ update_symbol_values(Elf *elf, GElf_Ehdr *ehdr,
>> > out why and also figure out whether the zero
>> > value should have
>> > been adjusted, after all.
>> > */
>> > - ASSERT(!(shdr_info[sym->st_shndx].shdr.sh_flags
>> > & SHF_ALLOC));
>> > ASSERT(shdr_info[i].shdr.sh_type ==
>> > SHT_SYMTAB);
>> > }
>> Like the -Werror changes that Alexander commented on, this change
>> circumvents the error output, but does not address the underlying
>> problem. If you remove an assertion, that at least needs a very
>> good explanation why the assertion was incorrect in the first
> With warm regards,
More information about the linaro-dev