On September 17, 2021 6:30:21 PM GMT+02:00, Martin Sebor <mse...@gmail.com> 
wrote:
>On 9/17/21 8:54 AM, Richard Earnshaw wrote:
>> 
>> 
>> On 16/09/2021 16:44, Martin Sebor via Gcc wrote:
>>> On 9/14/21 2:10 AM, Andrew Pinski via Gcc wrote:
>>>> Hi all,
>>>>    I am doing some bugzilla cleanup.  This includes disabling some
>>>> components and some versions for new bugs.
>>>> So far I have disabled versions before GCC 4 because we have not had a
>>>> report from someone for those versions in over 7 years.  I disabled
>>>> some versions which are about developmental branches which are
>>>> inactive too.
>>>> I also disabled the java, libgcj, fastjar, libmudflap, treelang and
>>>> libf2c components.
>>>>
>>>> I am in the process of moving away from having an inline-asm component
>>>> to an inline-asm keyword instead; this was suggested on IRC and I
>>>> agree.  After the current open bugs have moved away from the
>>>> inline-asm component, I will disable it also.
>>>>
>>>> If anyone else has any other suggestions that should be done, please
>>>> let me know and I will look into doing it.
>>>
>>> Re: Keywords: I find it useful to differentiate between two kinds of
>>> diagnostic bugs: false positives and false negatives (the latter for
>>> existing warnings that don't trigger when intended, as opposed to
>>> requests to enhance existing warnings or add new ones). I've been
>>> using Personal Tags for this but it might be useful to others as
>>> well.  If you agree and add the corresponding new keywords
>>> (false-positive and false-negative) I'll set them based on my Tags.
>>>
>>> One other suggestion: every once in a while someone asks if
>>> ice-on-invalid-code bugs apply to syntactically well-formed code that
>>> has undefined behavior (I don't believe it does).  It would help to
>>> clarify the Description for this Keyword (and, correspondingly, for
>>> ice-on-valid).  E.g., something like
>>>
>>> ice-on-invalid-code: ICE on code that is not syntactically valid.
>>> ice-on-valid-code: ICE on code that is syntactically valid.
>>>
>> 
>> What about syntactically valid but semantically invalid code?  I'd call 
>> that ICE-on-invalid as well.
>
>My understanding of the "valid" in the keyword is that it refers
>only to syntactic validity AKA well-formedness.

Yes. 

>But I only set the keyword to help with bug triage, I don't actually
>rely on it for anything myself.  The interpretation should be based
>on what its main consumers use it for.  I expect Richard Biener (CC'd)
>might be using it for his release planning activities and so might
>want to weigh in on this.

In fact syntactically valid code should not ICE and that's as important as 
semantically valid code not ICE. 

>Martin
>
>PS In bugs reported for my code in the middle end (often having to
>do with detecting semantically invalid AKA undefined code), I use
>ice-on-valid for well-formed test cases with undefined behavior.
>I set ice-on-invalid only for the less common cases of syntactically
>invalid code that somehow makes it into the middle-end or that's
>analyzed in the front end.  If this isn't right I'd like to know.

That's correct. 

Richard. 

Reply via email to