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: rs6000 load_toc


  else if (TARGET_ELF && !TARGET_AIX)
    abort ();
  else if (TARGET_32BIT)
    {
       rs6000_maybe_dead (emit_insn (gen_load_toc_aix_si (dest)));
     else
       rs6000_maybe_dead (emit_insn (gen_load_toc_aix_di (dest)));
    }

I would much rather have the abort() at the end if none of the cases are
handled.  The revised structure does not act as a self-test.

  else if (DEFAULT_ABI == ABI_AIX)
    {
      if (TARGET_32BIT)
         rs6000_maybe_dead (emit_insn (gen_load_toc_aix_si (dest)));
       else
         rs6000_maybe_dead (emit_insn (gen_load_toc_aix_di (dest)));
    }
  else
    abort();

Thanks, David


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