Age | Commit message (Collapse) | Author |
|
|
|
When GlobalOpt has determined that a GlobalVariable only ever has two values,
it would convert the GlobalVariable to a boolean, and introduce SelectInsts
at every load, to choose between the two possible values. These SelectInsts
introduce overhead and other unpleasantness.
This patch makes GlobalOpt just add range metadata to loads from such
GlobalVariables instead. This enables the same main optimization (as seen in
test/Transforms/GlobalOpt/integer-bool.ll), without introducing selects.
The main downside is that it doesn't get the memory savings of shrinking such
GlobalVariables, but this is expected to be negligible.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Incoming
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
This patch also lays the groundwork for the single-use instruction trick to
reduce the number of temporary variables.
|
|
|
|
|
|
struct argument
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
getPrimitiveSizeInBits works on integer types but not on pointer types.
This change makes the code more robust, and will help support the removal
of the pointer lowering pass.
This also requires removing const from some Type pointers. Types are
immutable, so the convention is that const is redundant.
|
|
|
|
|
|
|
|
|
|
Emscripten doesn't need to legalize pointers.
|
|
|
|
|
|
|