Re: [wwwdocs] Add no_sanitize attribute

2018-12-29 Thread Gerald Pfeifer
On Sun, 7 Oct 2018, Sandra Loosemore wrote:
> This is definitely an improvement.  One more tweak I'd make is
> 
> s/any of the options specified/the options provided as arguments to the
> attribute/

Thank you, Sandra.  I made that additional change.

On Mon, 8 Oct 2018, Martin Liška wrote:
> Yes, it's equivalent, thanks for the improvement.

Thanks for the reviews.  I (finally) applied this per the patch below;
sorry for the delay.

Gerald

Index: changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-8/changes.html,v
retrieving revision 1.97
diff -u -r1.97 changes.html
--- changes.html30 Sep 2018 14:38:54 -  1.97
+++ changes.html29 Dec 2018 21:08:04 -
@@ -251,11 +251,11 @@
 tests for pointer wrapping.
   
   
-New no_sanitize attribute has been added.  The attribute
-on functions is used to inform the compiler that it should
-not do sanitization of all options mentioned in sanitize option.
-A list of values acceptable by -fsanitize option can be
-provided.
+A new attribute no_sanitize can be applied to functions
+to instruct the compiler not to do sanitization of the options
+provided as arguments to the attribute. Acceptable values for
+no_sanitize match those acceptable by the
+-fsanitize command-line option.
 
 void __attribute__ ((no_sanitize ("alignment", "object-size")))
 f () { /* Do something. */; }

Re: [wwwdocs] Add no_sanitize attribute

2018-10-08 Thread Martin Liška
On 10/7/18 2:15 PM, Gerald Pfeifer wrote:
> On Wed, 18 Apr 2018, Martin Liška wrote:
>> I would like to mention the attribute in GCC 8 changes.
> 
> Index: htdocs/gcc-8/changes.html
> ===
>>   
>> New no_sanitize attribute has been added.  The attribute
>> on functions is used to inform the compiler that it should
>> not do sanitization of all options mentioned in sanitize option.
>> A list of values acceptable by -fsanitize option can be
>> provided.
>> 
>> void __attribute__ ((no_sanitize ("alignment", "object-size")))
>> f () { /* Do something. */; }
>> 
>>   
> 
> I've been struggling with this from a language perspective and 
> while not fully happy, believe/hope the below may improve things
> a bit.
> 
> Sandra, any recommendations from your end?
> 
> Martin, is my update technically acurate?

Hi.

Yes, it's equivalent, thanks for the improvement.

Martin

> 
> Index: gcc-8/changes.html
> ===
> RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-8/changes.html,v
> retrieving revision 1.97
> diff -u -r1.97 changes.html
> --- gcc-8/changes.html30 Sep 2018 14:38:54 -  1.97
> +++ gcc-8/changes.html7 Oct 2018 12:13:03 -
> @@ -251,11 +251,10 @@
>  tests for pointer wrapping.
>
>
> -New no_sanitize attribute has been added.  The attribute
> -on functions is used to inform the compiler that it should
> -not do sanitization of all options mentioned in sanitize option.
> -A list of values acceptable by -fsanitize option can be
> -provided.
> +A new attribute no_sanitize can be applied to functions
> +to instruct the compiler not to do sanitization of any of the options
> +specified.  Acceptable values for no_sanitize match those
> +acceptable by the -fsanitize command-line option.
>  
>  void __attribute__ ((no_sanitize ("alignment", "object-size")))
>  f () { /* Do something. */; }
> 
> Gerald
> 



Re: [wwwdocs] Add no_sanitize attribute

2018-10-07 Thread Sandra Loosemore

On 10/07/2018 06:15 AM, Gerald Pfeifer wrote:

On Wed, 18 Apr 2018, Martin Liška wrote:

I would like to mention the attribute in GCC 8 changes.


Index: htdocs/gcc-8/changes.html
===

   
 New no_sanitize attribute has been added.  The attribute
 on functions is used to inform the compiler that it should
 not do sanitization of all options mentioned in sanitize option.
 A list of values acceptable by -fsanitize option can be
 provided.
 
 void __attribute__ ((no_sanitize ("alignment", "object-size")))
 f () { /* Do something. */; }
 
   


I've been struggling with this from a language perspective and
while not fully happy, believe/hope the below may improve things
a bit.

Sandra, any recommendations from your end?

Martin, is my update technically acurate?

Index: gcc-8/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-8/changes.html,v
retrieving revision 1.97
diff -u -r1.97 changes.html
--- gcc-8/changes.html  30 Sep 2018 14:38:54 -  1.97
+++ gcc-8/changes.html  7 Oct 2018 12:13:03 -
@@ -251,11 +251,10 @@
  tests for pointer wrapping.


-New no_sanitize attribute has been added.  The attribute
-on functions is used to inform the compiler that it should
-not do sanitization of all options mentioned in sanitize option.
-A list of values acceptable by -fsanitize option can be
-provided.
+A new attribute no_sanitize can be applied to functions
+to instruct the compiler not to do sanitization of any of the options
+specified.  Acceptable values for no_sanitize match those
+acceptable by the -fsanitize command-line option.
  
  void __attribute__ ((no_sanitize ("alignment", "object-size")))
  f () { /* Do something. */; }



This is definitely an improvement.  One more tweak I'd make is

s/any of the options specified/the options provided as arguments to the 
attribute/


-Sandra


Re: [wwwdocs] Add no_sanitize attribute

2018-10-07 Thread Gerald Pfeifer
On Wed, 18 Apr 2018, Martin Liška wrote:
> I would like to mention the attribute in GCC 8 changes.

Index: htdocs/gcc-8/changes.html
===
>   
> New no_sanitize attribute has been added.  The attribute
> on functions is used to inform the compiler that it should
> not do sanitization of all options mentioned in sanitize option.
> A list of values acceptable by -fsanitize option can be
> provided.
> 
> void __attribute__ ((no_sanitize ("alignment", "object-size")))
> f () { /* Do something. */; }
> 
>   

I've been struggling with this from a language perspective and 
while not fully happy, believe/hope the below may improve things
a bit.

Sandra, any recommendations from your end?

Martin, is my update technically acurate?

Index: gcc-8/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-8/changes.html,v
retrieving revision 1.97
diff -u -r1.97 changes.html
--- gcc-8/changes.html  30 Sep 2018 14:38:54 -  1.97
+++ gcc-8/changes.html  7 Oct 2018 12:13:03 -
@@ -251,11 +251,10 @@
 tests for pointer wrapping.
   
   
-New no_sanitize attribute has been added.  The attribute
-on functions is used to inform the compiler that it should
-not do sanitization of all options mentioned in sanitize option.
-A list of values acceptable by -fsanitize option can be
-provided.
+A new attribute no_sanitize can be applied to functions
+to instruct the compiler not to do sanitization of any of the options
+specified.  Acceptable values for no_sanitize match those
+acceptable by the -fsanitize command-line option.
 
 void __attribute__ ((no_sanitize ("alignment", "object-size")))
 f () { /* Do something. */; }

Gerald


Re: [wwwdocs] Add no_sanitize attribute

2018-04-27 Thread Jeff Law
On 04/18/2018 06:22 AM, Martin Liška wrote:
> I would like to mention the attribute in GCC 8 changes.
OK.
jeff


[wwwdocs] Add no_sanitize attribute

2018-04-18 Thread Martin Liška
I would like to mention the attribute in GCC 8 changes.

Martin
Index: htdocs/gcc-8/changes.html
===
RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-8/changes.html,v
retrieving revision 1.67
diff -r1.67 changes.html
255a256,266
>   
> New no_sanitize attribute has been added.  The attribute
> on functions is used to inform the compiler that it should
> not do sanitization of all options mentioned in sanitize option.
> A list of values acceptable by -fsanitize option can be
> provided.
> 
> void __attribute__ ((no_sanitize ("alignment", "object-size")))
> f () { /* Do something. */; }
> 
>