[PATCH] Enable non-complex math builtins from C99 for Bionic

Michael Eager eager@eagerm.com
Sun Jul 28 16:37:00 GMT 2013


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 <joseph@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	 eager@eagercon.com
1960 Park Blvd., Palo Alto, CA 94306  650-325-8077



More information about the Gcc-patches mailing list