Displaying 4 results from an estimated 4 matches for "d60291".
Did you mean:
d60261
2019 Apr 04
2
[RFC] Should we add isa_or_null<>?
I've added a patch, temporarily using the name Chris suggested. Please let
me know what you think.
https://reviews.llvm.org/D60291
thanks...
don
On Thu, Apr 4, 2019 at 2:55 PM David Greene <dag at cray.com> wrote:
> Don Hinton <hintonda at gmail.com> writes:
>
> > > if (isa_or_null<T>(var)) {
> > > ...
> > > }
> > >
> > > at least according to what &...
2019 Apr 04
2
[RFC] Should we add isa_or_null<>?
...> ~Craig
>
>
> On Thu, Apr 4, 2019 at 4:16 PM Don Hinton via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
>
>> I've added a patch, temporarily using the name Chris suggested. Please
>> let me know what you think.
>>
>> https://reviews.llvm.org/D60291
>>
>> thanks...
>> don
>>
>> On Thu, Apr 4, 2019 at 2:55 PM David Greene <dag at cray.com> wrote:
>>
>>> Don Hinton <hintonda at gmail.com> writes:
>>>
>>> > > if (isa_or_null<T>(var)) {
>>> > >...
2019 Apr 05
2
[RFC] Should we add isa_or_null<>?
...t 4:16 PM Don Hinton via llvm-dev <
>>> llvm-dev at lists.llvm.org> wrote:
>>>
>>>> I've added a patch, temporarily using the name Chris suggested. Please
>>>> let me know what you think.
>>>>
>>>> https://reviews.llvm.org/D60291
>>>>
>>>> thanks...
>>>> don
>>>>
>>>> On Thu, Apr 4, 2019 at 2:55 PM David Greene <dag at cray.com> wrote:
>>>>
>>>>> Don Hinton <hintonda at gmail.com> writes:
>>>>>
>>>>&...
2019 Apr 04
2
[RFC] Should we add isa_or_null<>?
On Thu, Apr 4, 2019 at 10:30 AM David Greene <dag at cray.com> wrote:
> I don't think that's a correct replacement.
>
> if (var && isa<T>(var)) {
> ...
> }
>
> is not the same as:
>
> if (isa_or_null<T>(var)) {
> ...
> }
>
> at least according to what "isa_or_null" conveys to me.
>
This is the same