[Bug other/98027] New: CET support is documented to be explicitly enabled, however it's enabled by default

[Bug other/98027] New: CET support is documented to be explicitly enabled, however it's enabled by default

[doko at debian dot org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98027

            Bug ID: 98027
           Summary: CET support is documented to be explicitly enabled,
                    however it's enabled by default
           Product: gcc
           Version: 10.2.1
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: other
          Assignee: unassigned at gcc dot gnu.org
          Reporter: doko at debian dot org
  Target Milestone: ---

CET support is documented to be explicitly enabled, however it's enabled by
default.

install.texi reads:

@item --enable-cet
@itemx --disable-cet
Enable building target run-time libraries with control-flow
instrumentation, see @option{-fcf-protection} option.  When
@code{--enable-cet} is specified target libraries are configured
to add @option{-fcf-protection} and, if needed, other target
specific options to a set of building options.

The option is disabled by default.  When @code{--enable-cet=auto}
is used, it is enabled on Linux/x86 if target binutils
supports @code{Intel CET} instructions and disabled otherwise.
In this case the target libraries are configured to get additional
@option{-fcf-protection} option.

however it's still enabled when no --enable-/--disable-cet configure options is
given.

Please either update the documentation or the behavior. Seems to be the case
for the gcc-10 branch as well.

unreleated question: when do you configure with -fcf-protection -mshstk, and
when just with -fcf-protection?

[Bug target/98027] CET support is documented to be explicitly enabled, however it's enabled by default

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98027

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
           Keywords|                            |documentation
   Last reconfirmed|                            |2020-11-27
             Status|UNCONFIRMED                 |NEW
          Component|other                       |target
             Target|                            |x86_64-*-* i?86-*-*

--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> ---
I think support is autodetected now.  HJ, can you adjust the wording?

[Bug target/98027] CET support is documented to be explicitly enabled, however it's enabled by default

[cvs-commit at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98027

--- Comment #2 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by H.J. Lu <hjl@gcc.gnu.org>:

https://gcc.gnu.org/g:9df6c9c7a3936ff0a38a7066281842128cdd6914

commit r11-5499-g9df6c9c7a3936ff0a38a7066281842128cdd6914
Author: H.J. Lu <hjl.tools@gmail.com>
Date:   Fri Nov 27 05:39:56 2020 -0800

    INSTALL: Default to --enable-cet=auto

            PR other/98027
            * doc/install.texi: Default to --enable-cet=auto.

[Bug target/98027] CET support is documented to be explicitly enabled, however it's enabled by default

[hjl.tools at gmail dot com]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=98027

H.J. Lu <hjl.tools at gmail dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
   Target Milestone|---                         |11.0
         Resolution|---                         |FIXED

--- Comment #3 from H.J. Lu <hjl.tools at gmail dot com> ---
Fixed.