[WHLSL] Delete the 'restricted' keyword

[Myles C. Maxfield]

[WHLSL] Delete the 'restricted' keyword

[Myles C. Maxfield]

Created attachment 359215 [details]
Patch

[Robin Morisset]

Comment on attachment 359215 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=359215&action=review

LGTM

> Source/WebCore/Modules/webgpu/WHLSL/WHLSLResolveOverloadImpl.cpp:74
> +    if (minimumCostCandidates.size() == 1)

Maybe rename minimumCostCandidates, since I don't think there is a notion of cost anymore.

[Myles C. Maxfield]

(In reply to Robin Morisset from comment #2)
> Comment on attachment 359215 [details]
> Patch
> 
> View in context:
> https://bugs.webkit.org/attachment.cgi?id=359215&action=review
> 
> LGTM
> 
> > Source/WebCore/Modules/webgpu/WHLSL/WHLSLResolveOverloadImpl.cpp:74
> > +    if (minimumCostCandidates.size() == 1)
> 
> Maybe rename minimumCostCandidates, since I don't think there is a notion of
> cost anymore.

There is a notion of cost.

void foo(int) { ... }
void foo(uint) { ... }
foo(4) resolves to a different overload than foo(4u)

[Dean Jackson]

r=me too

[Myles C. Maxfield]

Committed r240097: <https://trac.webkit.org/changeset/240097>

[Radar WebKit Bug Importer]

<rdar://problem/47335266>