Bug 208738
| Summary: | Consider renaming moveConditionally... MacroAssembler emitters. | ||
|---|---|---|---|
| Product: | WebKit | Reporter: | Mark Lam <mark.lam> |
| Component: | JavaScriptCore | Assignee: | Nobody <webkit-unassigned> |
| Status: | NEW | ||
| Severity: | Normal | ||
| Priority: | P2 | ||
| Version: | WebKit Nightly Build | ||
| Hardware: | Unspecified | ||
| OS: | Unspecified | ||
| See Also: | https://bugs.webkit.org/show_bug.cgi?id=208731 | ||
Mark Lam
This moveConditionally... functions actually implement csel semantics instead of cmov. While the "moveConditionally" name can mean select, it is easily confused to mean cmov. Consider renaming them to variants of "select" or something less ambiguous.
| Attachments | ||
|---|---|---|
| Add attachment proposed patch, testcase, etc. |