diff mbox series

docs: add cavear for __builtin_cpu_supports

Message ID d0030ad6-b152-c307-7c0c-b2d447cb51fb@suse.cz
State Unresolved
Headers
Series docs: add cavear for __builtin_cpu_supports |

Checks

Context Check Description
snail/gcc-patch-check warning Git am fail log

Commit Message

Martin Liška Feb. 9, 2023, 10:35 a.m. UTC 
  Document that the function does not work correctly for old
VIA processors.

Ready to be installed?
Thanks,
Martin

	PR target/100758

gcc/ChangeLog:

	* doc/extend.texi: Document that the function
	does not work correctly for old VIA processors.
---
 gcc/doc/extend.texi | 3 +++
 1 file changed, 3 insertions(+)

Comments

Gerald Pfeifer Feb. 9, 2023, 10:43 a.m. UTC | #1 
On Thu, 9 Feb 2023, Martin Liška wrote:
> Document that the function does not work correctly for old VIA 
> processors.

Thanks for doing this, Martin!

> +Caveat: The built-in function does not work correctly on old VIA
> +processors (Centaur family).

Maybe say "This built-in function...", though that's a nuance. The patch 
is okay either way.

Gerald
Martin Liška Feb. 9, 2023, 10:50 a.m. UTC | #2 
On 2/9/23 11:43, Gerald Pfeifer wrote:
> On Thu, 9 Feb 2023, Martin Liška wrote:
>> Document that the function does not work correctly for old VIA 
>> processors.
> 
> Thanks for doing this, Martin!
> 
>> +Caveat: The built-in function does not work correctly on old VIA
>> +processors (Centaur family).
> 
> Maybe say "This built-in function...", though that's a nuance. The patch 
> is okay either way.

Thanks for review, I've just pushed it with the suggestion.

Martin

> 
> Gerald
diff mbox series

Patch

diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi
index 5a026c4b48c..2aedf6dfe69 100644
--- a/gcc/doc/extend.texi
+++ b/gcc/doc/extend.texi
@@ -22071,6 +22071,9 @@  else
      count = generic_countbits (n); //generic implementation.
   @}
 @end smallexample
+
+Caveat: The built-in function does not work correctly on old VIA
+processors (Centaur family).
 @end deftypefn
 
 The following built-in functions are made available by @option{-mmmx}.