[LLVMdev] [RFC] TileGX, a new backend for Tilera's many core processor
wendling at apple.com
Fri Mar 8 11:13:04 PST 2013
On Mar 8, 2013, at 2:31 AM, Jiong Wang <jiwang at tilera.com> wrote:
> On 03/08/2013 04:48 AM, Dmitri Gribenko wrote:
>> On Thu, Mar 7, 2013 at 6:33 PM, Jiong Wang <jiwang at tilera.com> wrote:
>>> Hi all,
>>> Updated the patches for TILE-Gx backend:
>>> 1. added initial regression tests for tilegx codegen.
>>> 2. added initial regression tests for MC Layer.
>>> 3. fixed those commenting style issues.
>>> please review, thanks.
>> This is a huge patch, and reviewing it in tar.gz is hard. To
>> facilitate review process, you can upload this to phabricator.
> OK, I am requesting for a phabricator account.
> one other things is, should I split the patch into several parts? like
> [PATCH 1/10]
> [PATCH 2/10]
> because clang patch & test-suite patch is relatively small, but llvm patch is nearly 10K line, still hard to review.
> if it is, will one patch for one file OK?
> What's the patch policy in llvm community ?
The LLVM community prefers small, self-contained patches. Please split up your patches into small chunks that can be easily reviewed. Keep in mind that the compiler needs to work after each pass goes in. :-)
More information about the llvm-dev