public inbox for gcc-rust@gcc.gnu.org
 help / color / mirror / Atom feed
From: Arthur Cohen <arthur.cohen@embecosm.com>
To: Richard Biener <richard.guenther@gmail.com>
Cc: gcc-patches@gcc.gnu.org, gcc-rust@gcc.gnu.org
Subject: Re: Rust frontend patches v3
Date: Tue, 15 Nov 2022 11:12:36 +0100	[thread overview]
Message-ID: <8af7c567-6634-8b83-1808-60d5d67943c4@embecosm.com> (raw)
In-Reply-To: <CAFiYyc0g21At-zGR8z0Rj+AC5SMsZ_uuncgRzKvnxrWJCHGhhg@mail.gmail.com>


[-- Attachment #1.1.1: Type: text/plain, Size: 5083 bytes --]

Hi Richard,

On 11/10/22 11:52, Richard Biener wrote:
> On Wed, Oct 26, 2022 at 10:16 AM <arthur.cohen@embecosm.com> wrote:
>>
>> This is the fixed version of our previous patch set for gccrs - We've adressed
>> the comments raised in our previous emails.
>>
>> This patch set does not contain any work that was not previously included, such
>> as closure support, the constant evaluator port, or the better implementation
>> of target hooks by Iain Buclaw. They will follow up in subsequent patch sets.
>>
>> Thanks again to Open Source Security, inc and Embecosm who have accompanied us
>> for this work.
>>
>> Many thanks to all of the contributors and our community, who made this
>> possible.
>>
>> A very special thanks to Philip Herron, without whose mentoring I would have
>> never been in a position to send these patches.
>>
>> You can see the current status of our work on our branch:
>> https://gcc.gnu.org/git/?p=gcc.git;a=shortlog;h=refs/heads/devel/rust/master
>>
>> The patch set contains the following:
> 
> Can you mark the patches that have been reviewed/approved?  Can you
> maybe either split the series or organize it in a way to separate the
> pieces touching common parts of GCC from the gcc/rust/ parts?
> Can you separate testsuite infrastructure from actual tests, can
> you mark/separate target specific changes?  And for those (then small)
> changes CC the appropriate maintainers?

Thanks a lot for all the feedback. I'll apply the required changes and 
make sure the patchset(s) are a bit easier to review.

All the best,

Arthur

> 
> Thanks,
> Richard.
> 
>> [PATCH Rust front-end v3 01/46] Use DW_ATE_UTF for the Rust 'char'
>> [PATCH Rust front-end v3 02/46] gccrs: Add nessecary hooks for a Rust
>> [PATCH Rust front-end v3 03/46] gccrs: Add Debug info testsuite
>> [PATCH Rust front-end v3 04/46] gccrs: Add link cases testsuite
>> [PATCH Rust front-end v3 05/46] gccrs: Add general compilation test
>> [PATCH Rust front-end v3 06/46] gccrs: Add execution test cases
>> [PATCH Rust front-end v3 07/46] gccrs: Add gcc-check-target
>> [PATCH Rust front-end v3 08/46] gccrs: Add Rust front-end base AST
>> [PATCH Rust front-end v3 09/46] gccrs: Add definitions of Rust Items
>> [PATCH Rust front-end v3 10/46] gccrs: Add full definitions of Rust
>> [PATCH Rust front-end v3 11/46] gccrs: Add Rust AST visitors
>> [PATCH Rust front-end v3 12/46] gccrs: Add Lexer for Rust front-end
>> [PATCH Rust front-end v3 13/46] gccrs: Add Parser for Rust front-end
>> [PATCH Rust front-end v3 14/46] gccrs: Add Parser for Rust front-end
>> [PATCH Rust front-end v3 15/46] gccrs: Add expansion pass for the
>> [PATCH Rust front-end v3 16/46] gccrs: Add name resolution pass to
>> [PATCH Rust front-end v3 17/46] gccrs: Add declarations for Rust HIR
>> [PATCH Rust front-end v3 18/46] gccrs: Add HIR definitions and
>> [PATCH Rust front-end v3 19/46] gccrs: Add AST to HIR lowering pass
>> [PATCH Rust front-end v3 20/46] gccrs: Add wrapper for make_unique
>> [PATCH Rust front-end v3 21/46] gccrs: Add port of FNV hash used
>> [PATCH Rust front-end v3 22/46] gccrs: Add Rust ABI enum helpers
>> [PATCH Rust front-end v3 23/46] gccrs: Add Base62 implementation
>> [PATCH Rust front-end v3 24/46] gccrs: Add implementation of Optional
>> [PATCH Rust front-end v3 25/46] gccrs: Add attributes checker
>> [PATCH Rust front-end v3 26/46] gccrs: Add helpers mappings canonical
>> [PATCH Rust front-end v3 27/46] gccrs: Add type resolution and trait
>> [PATCH Rust front-end v3 28/46] gccrs: Add Rust type information
>> [PATCH Rust front-end v3 29/46] gccrs: Add remaining type system
>> [PATCH Rust front-end v3 30/46] gccrs: Add unsafe checks for Rust
>> [PATCH Rust front-end v3 31/46] gccrs: Add const checker
>> [PATCH Rust front-end v3 32/46] gccrs: Add privacy checks
>> [PATCH Rust front-end v3 33/46] gccrs: Add dead code scan on HIR
>> [PATCH Rust front-end v3 34/46] gccrs: Add unused variable scan
>> [PATCH Rust front-end v3 35/46] gccrs: Add metadata ouptput pass
>> [PATCH Rust front-end v3 36/46] gccrs: Add base for HIR to GCC
>> [PATCH Rust front-end v3 37/46] gccrs: Add HIR to GCC GENERIC
>> [PATCH Rust front-end v3 38/46] gccrs: Add HIR to GCC GENERIC
>> [PATCH Rust front-end v3 39/46] gccrs: These are wrappers ported from
>> [PATCH Rust front-end v3 40/46] gccrs: Add GCC Rust front-end
>> [PATCH Rust front-end v3 41/46] gccrs: Add config-lang.in
>> [PATCH Rust front-end v3 42/46] gccrs: Add lang-spec.h
>> [PATCH Rust front-end v3 43/46] gccrs: Add lang.opt
>> [PATCH Rust front-end v3 44/46] gccrs: Add compiler driver
>> [PATCH Rust front-end v3 45/46] gccrs: Compiler proper interface
>> [PATCH Rust front-end v3 46/46] gccrs: Add README, CONTRIBUTING and
>>

-- 
Arthur Cohen <arthur.cohen@embecosm.com>

Toolchain Engineer

Embecosm GmbH

Geschäftsführer: Jeremy Bennett
Niederlassung: Nürnberg
Handelsregister: HR-B 36368
www.embecosm.de

Fürther Str. 27
90429 Nürnberg


Tel.: 091 - 128 707 040
Fax: 091 - 128 707 077

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3195 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 840 bytes --]

      reply	other threads:[~2022-11-15 10:09 UTC|newest]

Thread overview: 76+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-26  8:17 arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 01/46] Use DW_ATE_UTF for the Rust 'char' type arthur.cohen
2022-10-26  8:39   ` Jakub Jelinek
2022-10-30 15:22     ` Mark Wielaard
2022-10-30 17:25       ` Jakub Jelinek
2022-10-31 14:19       ` Tom Tromey
2022-11-15 20:37       ` Marc Poulhiès
2022-10-26  8:17 ` [PATCH Rust front-end v3 02/46] gccrs: Add nessecary hooks for a Rust front-end testsuite arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 03/46] gccrs: Add Debug info testsuite arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 04/46] gccrs: Add link cases testsuite arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 05/46] gccrs: Add general compilation test cases arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 06/46] gccrs: Add execution " arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 07/46] gccrs: Add gcc-check-target check-rust arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 08/46] gccrs: Add Rust front-end base AST data structures arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 09/46] gccrs: Add definitions of Rust Items in " arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 10/46] gccrs: Add full definitions of Rust " arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 11/46] gccrs: Add Rust AST visitors arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 12/46] gccrs: Add Lexer for Rust front-end arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 13/46] gccrs: Add Parser for Rust front-end pt.1 arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 14/46] gccrs: Add Parser for Rust front-end pt.2 arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 15/46] gccrs: Add expansion pass for the Rust front-end arthur.cohen
2022-11-10 10:49   ` Richard Biener
2022-11-15 11:35     ` Arthur Cohen
2022-11-16 15:49       ` Richard Biener
2022-11-17 14:22         ` Arthur Cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 16/46] gccrs: Add name resolution pass to " arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 17/46] gccrs: Add declarations for Rust HIR arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 18/46] gccrs: Add HIR definitions and visitor framework arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 19/46] gccrs: Add AST to HIR lowering pass arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 20/46] gccrs: Add wrapper for make_unique arthur.cohen
2022-10-26 20:56   ` David Malcolm
2022-10-26  8:17 ` [PATCH Rust front-end v3 21/46] gccrs: Add port of FNV hash used during legacy symbol mangling arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 22/46] gccrs: Add Rust ABI enum helpers arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 23/46] gccrs: Add Base62 implementation arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 24/46] gccrs: Add implementation of Optional arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 25/46] gccrs: Add attributes checker arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 26/46] gccrs: Add helpers mappings canonical path and lang items arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 27/46] gccrs: Add type resolution and trait solving pass arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 28/46] gccrs: Add Rust type information arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 29/46] gccrs: Add remaining type system transformations arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 30/46] gccrs: Add unsafe checks for Rust arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 31/46] gccrs: Add const checker arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 32/46] gccrs: Add privacy checks arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 33/46] gccrs: Add dead code scan on HIR arthur.cohen
2022-10-26  8:17 ` [PATCH Rust front-end v3 34/46] gccrs: Add unused variable scan arthur.cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 35/46] gccrs: Add metadata ouptput pass arthur.cohen
2022-10-26 21:04   ` David Malcolm
2022-10-27  8:09     ` Arthur Cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 36/46] gccrs: Add base for HIR to GCC GENERIC lowering arthur.cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 37/46] gccrs: Add HIR to GCC GENERIC lowering for all nodes arthur.cohen
2022-11-09 18:07   ` Richard Biener
2022-11-15 12:32     ` Arthur Cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 38/46] gccrs: Add HIR to GCC GENERIC lowering entry point arthur.cohen
2022-11-09 13:53   ` Richard Biener
2022-11-15 13:49     ` Arthur Cohen
2022-11-18 13:02       ` Richard Biener
2022-11-29 18:13         ` Arthur Cohen
2022-11-30  7:53           ` Richard Biener
2022-11-21  9:03     ` Thomas Schwinge
2022-10-26  8:18 ` [PATCH Rust front-end v3 39/46] gccrs: These are wrappers ported from reusing gccgo arthur.cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 40/46] gccrs: Add GCC Rust front-end Make-lang.in arthur.cohen
2022-10-27  8:14   ` Arthur Cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 41/46] gccrs: Add config-lang.in arthur.cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 42/46] gccrs: Add lang-spec.h arthur.cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 43/46] gccrs: Add lang.opt arthur.cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 44/46] gccrs: Add compiler driver arthur.cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 45/46] gccrs: Compiler proper interface kicks off the pipeline arthur.cohen
2022-10-26  8:18 ` [PATCH Rust front-end v3 46/46] gccrs: Add README, CONTRIBUTING and compiler logo arthur.cohen
2022-10-26 21:15 ` Rust frontend patches v3 David Malcolm
2022-10-28 11:48   ` Arthur Cohen
2022-10-28 12:31     ` Richard Biener
2022-10-28 13:06     ` David Malcolm
2022-10-28 15:20       ` Arthur Cohen
2022-10-28 16:29         ` David Malcolm
2022-11-10 10:52 ` Richard Biener
2022-11-15 10:12   ` Arthur Cohen [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8af7c567-6634-8b83-1808-60d5d67943c4@embecosm.com \
    --to=arthur.cohen@embecosm.com \
    --cc=gcc-patches@gcc.gnu.org \
    --cc=gcc-rust@gcc.gnu.org \
    --cc=richard.guenther@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).