Skip to content
This repository has been archived by the owner on Jun 28, 2022. It is now read-only.

Canonicalize on StartAppPC or DecodedPC, but not both. #78

Open
pgoodman opened this issue Dec 10, 2014 · 0 comments
Open

Canonicalize on StartAppPC or DecodedPC, but not both. #78

pgoodman opened this issue Dec 10, 2014 · 0 comments

Comments

@pgoodman
Copy link
Member

DecodedBasicBlock uses StartAppPC, but has no associated EndAppPC. It also has a StartCachePC, but having an end is meaningless due to fragment scheduling.

NativeInstruction uses DecodedPC and DecodedLength. This could equivalently be represented with bounding PCs, using StartAppPC and EndAppPC.

I think we should canonicalize on these two function names across blocks and instructions.

It's also worth thinking about if Start or Begin is the right prefix.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant