From: Tom N <nospam@codesniffer.com>
To: Jonathan Wakely <jwakely.gcc@gmail.com>
Cc: gcc-help <gcc-help@gcc.gnu.org>
Subject: Re: -fsanitize=undefined behavior
Date: Fri, 1 May 2020 13:13:37 -0400 (EDT) [thread overview]
Message-ID: <31960797.205994.1588353217751@email.ionos.com> (raw)
In-Reply-To: <CAH6eHdSoRe+rLnx5r=6ijxgsCPEzO7h3rEf_z2=Nc7wWVBWDNg@mail.gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1843 bytes --]
> On May 1, 2020 at 7:39 AM Jonathan Wakely <jwakely.gcc@gmail.com> wrote:
>
> On Thu, 30 Apr 2020 at 23:58, Tom N <nospam@codesniffer.com> wrote:>> > On April 30, 2020 at 5:57 PM Jonathan Wakely <jwakely.gcc@gmail.com> wrote:The man page says:"Unlike other similar options, -fsanitize=float-divide-by-zero is not enabled by -fsanitize=undefined" and "Unlike other similar options,-fsanitize=float-cast-overflow is not enabled by -fsanitize=undefined"So it seems reasonable to assume that all the other suboptions are enabled by -fsanitize=undefined, and if you want *all* of them then use -fsanitize=undefined -fsanitize=float-divide-by-zero-fsanitize=float-cast-overflow> Thanks for the tip. Any reason the description of -fsanitize=undefined doesn't include something like, "If the -fsanitize=undefined option is enabled all sub-options will be enabled unless otherwise specified." so it's clear?Probably because nobody has suggested doing that yet. Would you liketo propose a patch for the docs?
> The relevant place is here:https://gcc.gnu.org/git/?p=gcc.git;a=blob;f=gcc/doc/invoke.texi;h=527d362533ab503cee9568b27808177977a52ac8;hb=HEAD#l13541
I'd be happy to but unfortunately I'm not familiar with git. (I know...). But here's an old-style patch off 8.4 if it helps (attached too):
-----
--- gcc-8.4.0/gcc/doc/invoke.texi 2020-03-04 03:30:00.000000000 -0500
+++ gcc-8.4.0_new/gcc/doc/invoke.texi 2020-05-01 13:10:36.517390533 -0400
@@ -11404,7 +11404,8 @@
@opindex fsanitize=undefined
Enable UndefinedBehaviorSanitizer, a fast undefined behavior detector.
Various computations are instrumented to detect undefined behavior
-at runtime. Current suboptions are:
+at runtime. If this option is enabled all sub-options listed below will be
+enabled unless otherwise specified. Current suboptions are:
@table @gcctabopt
-----
[-- Attachment #2: sanitize_undefined_elaboration_gcc8.4.patch --]
[-- Type: text/x-patch, Size: 537 bytes --]
--- gcc-8.4.0/gcc/doc/invoke.texi 2020-03-04 03:30:00.000000000 -0500
+++ gcc-8.4.0_new/gcc/doc/invoke.texi 2020-05-01 13:10:36.517390533 -0400
@@ -11404,7 +11404,8 @@
@opindex fsanitize=undefined
Enable UndefinedBehaviorSanitizer, a fast undefined behavior detector.
Various computations are instrumented to detect undefined behavior
-at runtime. Current suboptions are:
+at runtime. If this option is enabled all sub-options listed below will be
+enabled unless otherwise specified. Current suboptions are:
@table @gcctabopt
next prev parent reply other threads:[~2020-05-01 17:13 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-04-30 20:30 Tom N
2020-04-30 21:57 ` Jonathan Wakely
2020-04-30 22:58 ` Tom N
2020-05-01 11:39 ` Jonathan Wakely
2020-05-01 17:13 ` Tom N [this message]
2020-05-01 18:17 ` Jonathan Wakely
2020-05-01 20:53 ` Tom N
2020-05-01 13:38 ` Segher Boessenkool
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=31960797.205994.1588353217751@email.ionos.com \
--to=nospam@codesniffer.com \
--cc=gcc-help@gcc.gnu.org \
--cc=jwakely.gcc@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).