Bug 154228
| Summary: | The result ValueRep shouldn't be in params[0]. StackmapGenerationParams should have a separate field for it. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Filip Pizlo <fpizlo> |
| Component: | JavaScriptCore | Assignee: | Saam Barati <saam> |
| Status: | NEW | ||
| Severity: | Normal | CC: | ggaren, saam |
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | All | ||
| OS: | All | ||
Filip Pizlo
Patch forthcoming.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |
Filip Pizlo
(In reply to comment #0)
> Patch forthcoming.
I didn't mean to say that. I'm not working on this right now.
Saam Barati
Sweet. I was thinking that this would be a good change to make.
Filip Pizlo
(In reply to comment #2)
> Sweet. I was thinking that this would be a good change to make.
Do you want to volunteer? :-)))
Saam Barati
(In reply to comment #3)
> (In reply to comment #2)
> > Sweet. I was thinking that this would be a good change to make.
>
> Do you want to volunteer? :-)))
I guess this is the last time I say I think a patch is a good idea ;)
I'm joking. Sure, I'll do it.
Filip Pizlo
(In reply to comment #4)
> (In reply to comment #3)
> > (In reply to comment #2)
> > > Sweet. I was thinking that this would be a good change to make.
> >
> > Do you want to volunteer? :-)))
> I guess this is the last time I say I think a patch is a good idea ;)
> I'm joking. Sure, I'll do it.
It's not a major priority either way, but it could be a fun clean-up. :-)