diff options
author | 2012-12-14 13:35:28 -0800 | |
---|---|---|
committer | 2013-02-06 12:16:33 -0800 | |
commit | 4ef3e45d7c6ec3c482a1a48f4df470811aa3cf0a (patch) | |
tree | 5d91ff23708048a2b453a5917dab82be1b545f91 /src/compiler/codegen/codegen.h | |
parent | f84f99fe3a944310fdfb6f17836079ac5c48c799 (diff) |
Compiler constant handling rework
In preparation for de-optimization, reworked the constant
handling mechanism. Also took advantage of knowledge of
constant operands (particularly for long operations).
Significant performance improvements for Mandelbrot
(~60 seconds to ~34 seconds). Minor improvements in other
benchmarks.
The new constant handling breaks two of the existing
optimization passes: "Skip Large Method" and "Load/Store
Elimization."
I don't intend to update the large method optimization
because it will be superceeded by the upcoming interpreter/
fingerprinting mechanism. Leaving the code in place for
now in order to compare compile-time improvements with
fingerprinting/interpret. All related code will be deleted
when that is complete.
The load/store elimination pass needs some rework to handle
uses of multiple-register loads and stores. It will be
updated & restored in a future CL.
Change-Id: Ia979abaf51b8ae81bbb0428031cbcea854625fac
Diffstat (limited to 'src/compiler/codegen/codegen.h')
-rw-r--r-- | src/compiler/codegen/codegen.h | 16 |
1 files changed, 13 insertions, 3 deletions
diff --git a/src/compiler/codegen/codegen.h b/src/compiler/codegen/codegen.h index 03ecb43171..901e5da6bb 100644 --- a/src/compiler/codegen/codegen.h +++ b/src/compiler/codegen/codegen.h @@ -236,8 +236,8 @@ class Codegen { int displacement, int r_dest, int r_dest_hi, OpSize size, int s_reg) = 0; virtual LIR* LoadConstantNoClobber(CompilationUnit* cu, int r_dest, int value) = 0; - virtual LIR* LoadConstantValueWide(CompilationUnit* cu, int r_dest_lo, int r_dest_hi, - int val_lo, int val_hi) = 0; + virtual LIR* LoadConstantWide(CompilationUnit* cu, int r_dest_lo, int r_dest_hi, + int64_t value) = 0; virtual LIR* StoreBaseDisp(CompilationUnit* cu, int rBase, int displacement, int r_src, OpSize size) = 0; virtual LIR* StoreBaseDispWide(CompilationUnit* cu, int rBase, int displacement, int r_src_lo, @@ -288,6 +288,10 @@ class Codegen { virtual bool IsUnconditionalBranch(LIR* lir) = 0; // Required for target - Dalvik-level generators. + virtual bool GenArithImmOpLong(CompilationUnit* cu, Instruction::Code opcode, RegLocation rl_dest, + RegLocation rl_src1, RegLocation rl_src2) = 0; + virtual void GenMulLong(CompilationUnit* cu, RegLocation rl_dest, RegLocation rl_src1, + RegLocation rl_src2) = 0; virtual bool GenAddLong(CompilationUnit* cu, RegLocation rl_dest, RegLocation rl_src1, RegLocation rl_src2) = 0; virtual bool GenAndLong(CompilationUnit* cu, RegLocation rl_dest, RegLocation rl_src1, @@ -349,6 +353,9 @@ class Codegen { RegLocation rl_index, RegLocation rl_dest, int scale) = 0; virtual void GenArrayPut(CompilationUnit* cu, int opt_flags, OpSize size, RegLocation rl_array, RegLocation rl_index, RegLocation rl_src, int scale) = 0; + virtual bool GenShiftImmOpLong(CompilationUnit* cu, Instruction::Code opcode, + RegLocation rl_dest, RegLocation rl_src1, + RegLocation rl_shift) = 0; // Required for target - single operation generators. virtual LIR* OpUnconditionalBranch(CompilationUnit* cu, LIR* target) = 0; @@ -381,7 +388,10 @@ class Codegen { virtual void OpRegCopyWide(CompilationUnit* cu, int dest_lo, int dest_hi, int src_lo, int src_hi) = 0; virtual void OpTlsCmp(CompilationUnit* cu, int offset, int val) = 0; - virtual bool InexpensiveConstant(int reg, int value) = 0; + virtual bool InexpensiveConstantInt(int32_t value) = 0; + virtual bool InexpensiveConstantFloat(int32_t value) = 0; + virtual bool InexpensiveConstantLong(int64_t value) = 0; + virtual bool InexpensiveConstantDouble(int64_t value) = 0; // Temp workaround void Workaround7250540(CompilationUnit* cu, RegLocation rl_dest, int value); |