On Tue, Nov 17, 2015 at 12:14 AM, Mike Stump wrote:
> On Nov 16, 2015, at 1:52 PM, Richard Sandiford wrote:
>>
>> Yeah. Kenny was adamant that for wide-int we should have an UNSIGNED/SIGNED
>
> Yeah, you can blame me. I think (, UNSIGNED) conveys more than (,true) or
> (,false). The sad part
On Nov 16, 2015, at 1:52 PM, Richard Sandiford wrote:
>
> Yeah. Kenny was adamant that for wide-int we should have an UNSIGNED/SIGNED
Yeah, you can blame me. I think (, UNSIGNED) conveys more than (,true) or
(,false). The sad part is, this has always been true.
> enum rather than a boolean
Jeff Law writes:
Boolean params are best avoided if possible, IMO. In this case,
it seems this could instead be a new wrapper function, like:
>>> This hasn't been something we've required for GCC.I've come across
>>> this recommendation a few times over the last several months as I
On 11/16/2015 01:15 PM, Pedro Alves wrote:
Hi Jeff,
(Sorry I didn't reply sooner, I was OOO.)
No worries.
Boolean params are best avoided if possible, IMO. In this case,
it seems this could instead be a new wrapper function, like:
This hasn't been something we've required for GCC.I've
Hi Jeff,
(Sorry I didn't reply sooner, I was OOO.)
On 11/08/2015 11:17 PM, Jeff Law wrote:
> On 11/07/2015 07:31 AM, Pedro Alves wrote:
>> Hi Richard,
>>
>> Passerby comment below.
>>
>> On 11/07/2015 01:21 PM, Richard Sandiford wrote:
>>> -/* Lookup the identifier ID. */
>>> +/* Lookup the iden
On Mon, Nov 9, 2015 at 12:17 AM, Jeff Law wrote:
> On 11/07/2015 07:31 AM, Pedro Alves wrote:
>>
>> Hi Richard,
>>
>> Passerby comment below.
>>
>> On 11/07/2015 01:21 PM, Richard Sandiford wrote:
>>>
>>> -/* Lookup the identifier ID. */
>>> +/* Lookup the identifier ID. Allow "null" if ALLOW_NU
On 11/07/2015 07:31 AM, Pedro Alves wrote:
Hi Richard,
Passerby comment below.
On 11/07/2015 01:21 PM, Richard Sandiford wrote:
-/* Lookup the identifier ID. */
+/* Lookup the identifier ID. Allow "null" if ALLOW_NULL. */
id_base *
-get_operator (const char *id)
+get_operator (const char
Hi Richard,
Passerby comment below.
On 11/07/2015 01:21 PM, Richard Sandiford wrote:
> -/* Lookup the identifier ID. */
> +/* Lookup the identifier ID. Allow "null" if ALLOW_NULL. */
>
> id_base *
> -get_operator (const char *id)
> +get_operator (const char *id, bool allow_null = false)
>
