diff options
| author | Dmitry Petrochenko <dmitry.petrochenko@intel.com> | 2014-05-17 01:02:18 +0700 |
|---|---|---|
| committer | Dmitry Petrochenko <dmitry.petrochenko@intel.com> | 2014-06-07 01:28:05 +0000 |
| commit | 58994cdb00b323339bd83828eddc53976048006f (patch) | |
| tree | a1ec67c5b04cf99439619bf972a58b6d46dd2823 /compiler/dex/frontend.cc | |
| parent | 9529d6273777ee297a8aa7513e8172775f0496df (diff) | |
| download | android_art-58994cdb00b323339bd83828eddc53976048006f.tar.gz android_art-58994cdb00b323339bd83828eddc53976048006f.tar.bz2 android_art-58994cdb00b323339bd83828eddc53976048006f.zip | |
x86_64: Hard Float ABI support in QCG
This patch shows our efforts on resolving the ART limitations:
- passing "float"/"double" arguments via FPR
- passing "long" arguments via single GPR, not pair
- passing more than 3 agruments via GPR.
Work done:
- Extended SpecialTargetRegister enum with kARG4, kARG5, fARG4..fARG7.
- Created initial LoadArgRegs/GenDalvikX/FlushIns version in X86Mir2Lir.
- Unlimited number of long/double/float arguments support
- Refactored (v2)
Change-Id: I5deadd320b4341d5b2f50ba6fa4a98031abc3902
Signed-off-by: Serguei Katkov <serguei.i.katkov@intel.com>
Signed-off-by: Dmitry Petrochenko <dmitry.petrochenko@intel.com>
Signed-off-by: Chao-ying Fu <chao-ying.fu@intel.com>
Signed-off-by: Mark Mendell <mark.p.mendell@intel.com>
Diffstat (limited to 'compiler/dex/frontend.cc')
| -rw-r--r-- | compiler/dex/frontend.cc | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/compiler/dex/frontend.cc b/compiler/dex/frontend.cc index 5b9c763af6..547c0f6b30 100644 --- a/compiler/dex/frontend.cc +++ b/compiler/dex/frontend.cc @@ -707,7 +707,7 @@ int x86_64_support_list[] = { // which has problems with long, float, double constexpr char arm64_supported_types[] = "ZBSCILVJFD"; // (x84_64) We still have troubles with compiling longs/doubles/floats -constexpr char x86_64_supported_types[] = "ZBSCILV"; +constexpr char x86_64_supported_types[] = "ZBSCILVJFD"; // TODO: Remove this when we are able to compile everything. static bool CanCompileShorty(const char* shorty, InstructionSet instruction_set) { @@ -718,7 +718,7 @@ static bool CanCompileShorty(const char* shorty, InstructionSet instruction_set) // 1 is for the return type. Currently, we only accept 2 parameters at the most. // (x86_64): For now we have the same limitation. But we might want to split this // check in future into two separate cases for arm64 and x86_64. - if (shorty_size > (1 + 2)) { + if ((shorty_size > (1 + 2)) && (instruction_set != kX86_64)) { return false; } |