On 07/27/13 15:18, Alexander Ivchenko wrote:
Hi Joseph, thanks for your comments.

I updated the patch:


2013/7/9 Joseph S. Myers <jos...@codesourcery.com>:


* It looks rather like microblaze*-*-* don't use elfos.h, so meaning
semantics aren't preserved for those (non-Linux) targets either.  Now, I
don't know if there's a good reason for not using that file (ask the
architecture maintainer), but in any case semantics should be preserved.


I don't know why microblaze does not include elfos.h.   It looks like
it should, to be consistent with other targets.  This would require some
cleanup and verification.

Your patch adds the following to microblaze.h, duplicating the change
to elfos.h:
+/* microblaze-unknown-elf target has no support of C99 runtime */
+#undef TARGET_LIBC_HAS_FUNCTION
+#define TARGET_LIBC_HAS_FUNCTION no_c99_libc_has_function

I'm assuming that this means that no other change to microblaze is
needed and the question about elfos.h is moot.

--
Michael Eager    ea...@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077

Reply via email to