[cfe-dev] [LLVMdev] About commit TILE-Gx backend to community repository and default disabled
jiwang at tilera.com
Sat Mar 23 09:34:08 PDT 2013
于 2013/3/24 0:07, Joerg Sonnenberger 写道:
> On Sat, Mar 23, 2013 at 04:59:49PM +0100, Tobias Grosser wrote:
>> With Chris's reply you also got the general OK to upstream the
>> patch, _AFTER_ the actual patches have been reviewed.
> One thing I was asking on IRC for is whether it makes sense for new
> backends to be committable in incremental steps. Especially for a
> complete backend like this, e.g. the asm parser would make a reasonable
> and useful part to get in and limit the amount of code to be reviewed at
Hi Tobias & Joerg,
I understand your concern.
yes, Asm Parser is a module which could be committed individually, but I
think the code
for AsmParser is relatively small and highly target dependent, a full
regression test which
cover all instructions could largly gurantee the correctness.
I know contributors have their own works. For TILE-Gx backend, it's good
if any contributor could have time to finish a full set review, I have
been wating for several weeks.
I personly think it's not premature to commit TILE-Gx backend into
mainline and default disable. Because:
1. TILE-Gx finished the regression test & test-suite test.
2. TILE-Gx buildbot will be setup, so it will be convinent for both
community & Tilera company to monitor the status.
So should I keep waiting the full set review? or it's OK to commit and
> one point.
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
More information about the cfe-dev