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: [PATCH] Add sanopt support for UBSAN_PTR.

From: Martin Sebor <msebor at gmail dot com>
To: Martin Liška <mliska at suse dot cz>, gcc-patches at gcc dot gnu dot org
Cc: Jakub Jelinek <jakub at redhat dot com>
Date: Fri, 6 Oct 2017 11:13:00 -0600
Subject: Re: [PATCH] Add sanopt support for UBSAN_PTR.
Authentication-results: sourceware.org; auth=none
References: <83817c1e-3138-13d7-1dde-6f72ca8f4ab9@suse.cz> <c871ef9b-280f-ed6c-6402-74eec03d4e05@gmail.com> <731f919c-2024-fdcd-bf20-5d49036c75d9@suse.cz> <375d9311-a670-497a-3f3f-d9e978f2ca32@gmail.com>

This looks like an impressive improvement!  FWIW, I've been
meaning to look into similar opportunities mentioned in bug
79265.


Hi. Thank you very much for feedback. If you want I can help with the PR?


I belatedly realized I didn't answer one of your questions (or
got an answer to one of mine :) so let me correct my oversight.

If you have the cycles to also tackle bug 79265 that would be
great. (I don't think of the bugs I file as my projects so if
one strikes you as interesting please feel free to grab it.)


Would making use of get_range_info() make sense here to detect
and eliminate even more cases?


I am curious what you think of this idea.

Martin

References:
- [PATCH] Add sanopt support for UBSAN_PTR.
  - From: Martin Liška
- Re: [PATCH] Add sanopt support for UBSAN_PTR.
  - From: Martin Sebor
- Re: [PATCH] Add sanopt support for UBSAN_PTR.
  - From: Martin Liška
- Re: [PATCH] Add sanopt support for UBSAN_PTR.
  - From: Martin Sebor

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