[cfe-dev] [LLVMdev] RFC: TileGX, a new backend for Tilera's many core processor
jiwang at tilera.com
Fri Mar 1 06:52:37 PST 2013
On 03/01/2013 10:42 PM, Hal Finkel wrote:
> As some of the llvm modules are in active development, for example MC
> Layer, we want to return code to community repository first, so that
> it will be easy to keep pace with llvm main tree.
> I think this makes sense; but my impression is that the community will want a clear idea that this will be maintained and improved for the foreseeable future.
sure, tilegx will be actively maintained & improved.
>> 2. There are no regression tests -- using the test-suite is obviously
>> useful, but targeted regression tests are essential. yes, regression
>> tests will be added later.
> I don't speak for everybody, but I'm pretty sure that you won't get an okay to commit this upstream without regression tests. We need a good degree of coverage in test/CodeGen/Tile (you can probably look at one of the other more-recent targets, like AArch64, to get an idea of what is required).
thanks for pointting this out, I will add regression tests, then
re-send the patch.
before this, please feel free to give comments and suggestions
on the existed patches.
>> You say that there are unexpected failures in the regression tests
>> and in the test suite. Do you understand these? yes, I have done
>> investigation on these failures.
>> regression (44 failures)
>> 20: caused JITTests failures, we lack old jit support
>> the other 20 are mostly under Clang:: Tooling (these testcases seems
>> to be added in this week) & some debuginfo testcases, I will clean
>> up them later.
>> test-suite (17 failures)
>> 1 of them are compile stage failures
>> the reason is tilegx do not support some c99 float rounding &
>> we lack the support of some flag like FE_DIVBYZERO etc, for
>> 16 of them are runtime failures.
>> actually, I compare all these failures with our gcc output, it's the
>> nearly all of them are caused by float point precision issue, for
>> -6|496234.333333333 <=== expected output & x86_64 result
>> +6|496234.333333334 <=== tilegx gcc/llvm result
>> TILE-Gx is a VLIW architecture with 64-bit registers, 64-bit address
>> and 64-bit instructions. TILE-Gx has load-store architecture ISAs.
>> More information on the architectures is available at
>> http://www.tilera.com/scm/docs/index.html .
>> the attached patches contains the following main features for tilegx
>> 1. general function.
>> 2. PIC/TLS/JumpTable.
>> 3. Instructoin Bundling for VLIW.
>> 4. Basic support for Asm Parser.
>> 5. MC Layer (aware of VLIW), MCJIT support.
>> I've run the regression test and standalone test-suite natively on
>> TILE-Gx silicon. The
>> test results are:
>> Expected Passes : 13218
>> Expected Failures : 78
>> Unsupported Tests : 68
>> Unexpected Failures: 44
>> Expected Passes : 949
>> Unexpected Failures: 17
>> the llvm patch is against:
>> commit 5e812139690ce077d568ef6559992b2cf74eb536
>> Author: Evgeniy Stepanov <eugeni.stepanov at gmail.com> Date: Thu Feb 28
>> 11:25:14 2013 +0000
>> [msan] Implement sanitize_memory attribute.
>> the clang patch is against:
>> commit a4d4621b206f941cc58d9d0bc7c67a8e705c9d49
>> Author: Daniel Jasper <djasper at google.com> Date: Thu Feb 28 11:05:57
>> 2013 +0000
>> Improve formatting of #defines.
>> the test-suite patch is against:
>> commit 8c1ab3b660e67b74421d657408167b1345188f8d
>> Author: Duncan Sands <baldrick at free.fr> Date: Sun Feb 17 15:21:11
>> 2013 +0000
>> Use LLVMCC_EMITIR_FLAG rather than -emit-llvm
>> please review, looking forward to your feedback.
>> Tilera Corporation.
>> LLVM Developers mailing list LLVMdev at cs.uiuc.edu
>> Jiong. Wang
>> Tilera Corporation.
More information about the cfe-dev