-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Labels
ImprovementThis change improves the code's behaviourThis change improves the code's behaviourOptimisationThis change somehow improves optimisationThis change somehow improves optimisationPatchThis change is an unobtrusive changeThis change is an unobtrusive change
Description
This change would have two potential benefits:
- The undefined behaviour eminating from
<<with a negative left hand value would be replaced with the well-defined behaviour of an unsigned type. - Theoretically this should generate less code because the code would no longer be using unnecessary excess bytes during the shift operation. Whether this is true in practice would have to be tested - the compiler may already be optimising this.
The code should be functionally equivalent, hence this should be a 'patch' change.
Metadata
Metadata
Assignees
Labels
ImprovementThis change improves the code's behaviourThis change improves the code's behaviourOptimisationThis change somehow improves optimisationThis change somehow improves optimisationPatchThis change is an unobtrusive changeThis change is an unobtrusive change