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)?

> 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.

Gerald

Reply via email to