[cfe-dev] [LLVMdev] About commit TILE-Gx backend to community repository and default disabled

Jiong Wang 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 
default disabled?

---
Regards,
Jiong
Tilera Corporation.
> one point.
>
> Joerg
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the cfe-dev mailing list