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: Support for Runtime CPU type detection via builtins (issue5754058)


On Sun, Apr 8, 2012 at 6:17 AM, Gerald Pfeifer <gerald@pfeifer.com> wrote:
> On Thu, 29 Mar 2012, Sriraman Tallam wrote:
>> Hi,
>>
>> ? ? I have made a new patch to only have two builtins :
>>
>> * __builtin_cpu_is ("<CPUNAME>")
>> * __builtin_cpu_supports ("<FEATURE>")
>>
>> apart from the cpu init builtin, __builtin_cpu_init.
>
> I don't see any .texi file as part of this change. ?Shouldn't this
> be documented (and also added to the release notes gcc-4.8/changes.html)?

I modified extend.texi for documenting the builtins. I guess you
missed it. I am reattaching the patch just in case.

>
>> List of CPU names :
>>
>> * "amd"
>> * "intel"
>
> Are company names really suitable here? ?Intel is also still producing
> ia64 aka Itanium, and in the future AMD might do some ARM-based designs.
> AMD64 and Intel64 might work.

This is basically the vendor info from CPUID. I dont mind changing it
but I do not understand why Intel or AMD is unsuitable.

>
> Gerald

Attachment: cpu_runtime_patch.txt
Description: Text document


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