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 new built-in: __builtin_unreachable()


Richard Guenther <richard.guenther@gmail.com> writes:

>>>> I have added a new command line flag (-funreachable-traps) that causes
>>>> __builtin_unreachable() to be expanded exactly as if it were a
>>>> __builtin_trap().
>>>
>>> I wouldn't add a new flag to do this. ÂIn fact preprocessor wrapping
>>> around the two builtins can do this. ÂOr was there a particular reason?
>>>
>>
>> My reason was that I thought iant requested it. ÂAlthough
>> '-D__builtin_unreachable=__builtin_trap' might accomplish the same thing
>> without polluting the flag namespace.
>
> Ian may want to clarify here.  I have my don't-add-new-parameters hat on,
> so without a compelling reason I would not add it.

I said a command line option, but I probably didn't really mean it.
What I want is some way to change it, for debugging purposes.  But Richi
is right: this can be handled in the preprocessor, one way or another.
So never mind about the command line option.  Sorry about that.

Ian


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