Bug 263560 - [JSC][armv7] MacroAssemblerARMv7::transfer32 should invalidate dataTempRegister
Summary: [JSC][armv7] MacroAssemblerARMv7::transfer32 should invalidate dataTempRegister
Status: RESOLVED INVALID
Alias: None
Product: WebKit
Classification: Unclassified
Component: JavaScriptCore (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Nobody
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-10-23 14:18 PDT by Joseph Griego
Modified: 2023-10-23 14:26 PDT (History)
0 users

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Joseph Griego 2023-10-23 14:18:39 PDT 
Exactly as title; patch forthcoming
Comment 1 Joseph Griego 2023-10-23 14:24:37 PDT 
Actually
Comment 2 Joseph Griego 2023-10-23 14:26:36 PDT 
Actually, this is pretty pervasive in the armv7 macroassembler; will look and see what, if any patch, we should have to address.

In principle, AIUI, not invalidating the cached value for the data/address temps when clobbering them could misbehave badly, but in practice I imagine it's difficult/near-impossible to trigger.