[clang] 3c604e9 - Update the clang and clang-tools-extra code owners files

Aaron Ballman via cfe-commits cfe-commits at lists.llvm.org
Tue Sep 6 05:30:04 PDT 2022 

Author: Aaron Ballman
Date: 2022-09-06T08:28:03-04:00
New Revision: 3c604e9f15606fa395b088ca341a232ffda2bb7d

URL: https://github.com/llvm/llvm-project/commit/3c604e9f15606fa395b088ca341a232ffda2bb7d
DIFF: https://github.com/llvm/llvm-project/commit/3c604e9f15606fa395b088ca341a232ffda2bb7d.diff

LOG: Update the clang and clang-tools-extra code owners files

This also converts the Clang code owners file from a flat text file to
an RST file that is linked in to the rest of our documentation.

The RFC for this can be found at:
https://discourse.llvm.org/t/rfc-proposed-changes-to-clangs-code-ownership/

Differential Revision: https://reviews.llvm.org/D132550

Added: 
    clang/CodeOwners.rst
    clang/docs/CodeOwners.rst

Modified: 
    clang-tools-extra/CODE_OWNERS.TXT
    clang/docs/index.rst

Removed: 
    clang/CODE_OWNERS.TXT


################################################################################
diff  --git a/clang-tools-extra/CODE_OWNERS.TXT b/clang-tools-extra/CODE_OWNERS.TXT
index 3dc3ddd7c78d..4cf80aa2b0b8 100644
--- a/clang-tools-extra/CODE_OWNERS.TXT
+++ b/clang-tools-extra/CODE_OWNERS.TXT
@@ -4,26 +4,27 @@ also the gatekeepers for their part of Clang, with the final word on what goes
 in or not.
 
 The list is sorted by surname and formatted to allow easy grepping and
-beautification by scripts.  The fields are: name (N), email (E), web-address
-(W), PGP key ID and fingerprint (P), description (D), and snail-mail address
-(S).
+beautification by scripts.  The fields are: name (N), email (E), Phabricator
+handle (H), and description (D).
 
 N: Aaron Ballman
 E: aaron at aaronballman.com
+H: aaron.ballman
 D: clang-query
 
-N: Manuel Klimek
-E: klimek at google.com
-D: clang-rename, all parts of clang-tools-extra not covered by someone else
-
-N: Alexander Kornienko
-E: alexfh at google.com
-D: clang-tidy
-
 N: Julie Hockett
 E: juliehockett at google.com
 D: clang-doc
 
+N: Nathan James
+E: n.james93 at hotmail.co.uk
+H: njames93
+D: clang-tidy
+
+N: Manuel Klimek
+E: klimek at google.com
+D: clang-rename, all parts of clang-tools-extra not covered by someone else
+
 N: Sam McCall
 E: sammccall at google.com
 D: clangd

diff  --git a/clang/CODE_OWNERS.TXT b/clang/CODE_OWNERS.TXT
deleted file mode 100644
index 740aeeb7125b..000000000000
--- a/clang/CODE_OWNERS.TXT
+++ /dev/null
@@ -1,66 +0,0 @@
-This file is a list of the people responsible for ensuring that patches for a
-particular part of Clang are reviewed, either by themself or by someone else.
-They are also the gatekeepers for their part of Clang, with the final word on
-what goes in or not.
-
-The list is sorted by surname and formatted to allow easy grepping and
-beautification by scripts.  The fields are: name (N), email (E), web-address
-(W), PGP key ID and fingerprint (P), description (D), and snail-mail address
-(S).
-
-N: Alexey Bader
-E: alexey.bader at intel.com
-D: SYCL support
-
-N: Aaron Ballman
-E: aaron at aaronballman.com
-D: Clang attributes
-
-N: Alexey Bataev
-E: a.bataev at hotmail.com
-D: OpenMP support
-
-N: Chandler Carruth
-E: chandlerc at gmail.com
-E: chandlerc at google.com
-D: CMake, library layering
-
-N: Eric Christopher
-E: echristo at gmail.com
-D: Debug Information, inline assembly
-
-N: Devin Coughlin
-E: dcoughlin at apple.com
-D: Clang Static Analyzer
-
-N: Doug Gregor
-E: dgregor at apple.com
-D: Emeritus owner
-
-N: Reid Kleckner
-E: rnk at google.com
-D: Microsoft C++ ABI compatibility and general Windows support
-
-N: Manuel Klimek
-E: klimek at google.com
-D: AST matchers, LibTooling
-
-N: Anton Korobeynikov
-E: anton at korobeynikov.info
-D: Exception handling, Windows codegen, ARM EABI
-
-N: John McCall
-E: rjmccall at apple.com
-D: Clang LLVM IR generation
-
-N: Brad Smith
-E: brad at comstyle.com
-D: OpenBSD driver
-
-N: Richard Smith
-E: richard at metafoo.co.uk
-D: All parts of Clang not covered by someone else
-
-N: Anastasia Stulova
-E: anastasia.stulova at arm.com
-D: OpenCL support

diff  --git a/clang/CodeOwners.rst b/clang/CodeOwners.rst
new file mode 100644
index 000000000000..70662a6a580d
--- /dev/null
+++ b/clang/CodeOwners.rst
@@ -0,0 +1,256 @@
+=================
+Clang Code Owners
+=================
+
+This file is a list of the people responsible for ensuring that patches for a
+particular part of Clang are reviewed, either by themself or by someone else.
+They are also the gatekeepers for their part of Clang, with the final word on
+what goes in or not.
+
+.. contents::
+   :depth: 2
+   :local:
+
+Current Code Owners
+===================
+The following people are the active code owners for the project. Please reach
+out to them for code reviews, questions about their area of expertise, or other
+assistance.
+
+All parts of Clang not covered by someone else
+----------------------------------------------
+| Aaron Ballman
+| aaron\@aaronballman.com (email), aaron.ballman (Phabricator), AaronBallman (GitHub)
+
+
+Contained Components
+--------------------
+These code owners are responsible for particular high-level components within
+Clang that are typically contained to one area of the compiler.
+
+AST matchers
+~~~~~~~~~~~~
+| Manuel Klimek
+| klimek\@google.com (email), klimek (Phabricator), r4nt (GitHub)
+
+
+Clang LLVM IR generation
+~~~~~~~~~~~~~~~~~~~~~~~~
+| John McCall
+| rjmccall\@apple.com (email), rjmccall (Phabricator), rjmccall (GitHub)
+
+| Eli Friedman
+| efriedma\@quicinc.com (email), efriedma (Phabricator), efriedma-quic (GitHub)
+
+| Anton Korobeynikov
+| anton\@korobeynikov.info (email), asl (Phabricator), asl (GitHub)
+
+
+Analysis & CFG
+~~~~~~~~~~~~~~
+| Dmitri Gribenko
+| gribozavr\@gmail.com (email), gribozavr (Phabricator), gribozavr (GitHub)
+
+| Yitzhak Mandelbaum
+| yitzhakm\@google.com (email), ymandel (Phabricator), ymand (GitHub)
+
+| Stanislav Gatev
+| sgatev\@google.com (email), sgatev (Phabricator), sgatev (GitHub)
+
+
+Modules & serialization
+~~~~~~~~~~~~~~~~~~~~~~~
+| Chuanqi Xu
+| yedeng.yd\@linux.alibaba.com (email), ChuanqiXu (Phabricator), ChuanqiXu9 (GitHub)
+
+| Michael Spencer
+| bigcheesegs\@gmail.com (email), Bigcheese (Phabricator), Bigcheese (GitHub)
+
+
+Templates
+~~~~~~~~~
+| Erich Keane
+| erich.keane\@intel.com (email), ErichKeane (Phabricator), erichkeane (GitHub)
+
+
+Debug information
+~~~~~~~~~~~~~~~~~
+| Eric Christopher
+| echristo\@gmail.com (email), echristo (Phabricator), echristo (GitHub)
+
+
+Exception handling
+~~~~~~~~~~~~~~~~~~
+| Anton Korobeynikov
+| anton\@korobeynikov.info (email), asl (Phabricator), asl (GitHub)
+
+
+Clang static analyzer
+~~~~~~~~~~~~~~~~~~~~~
+| Artem Dergachev
+| adergachev\@apple.com (email), NoQ (Phabricator), haoNoQ (GitHub)
+
+| Gábor Horváth
+| xazax.hun\@gmail.com (email), xazax.hun (Phabricator), Xazax-hun (GitHub)
+
+
+Compiler options
+~~~~~~~~~~~~~~~~
+| Jan Svoboda
+| jan_svoboda\@apple.com (email), jansvoboda11 (Phabricator), jansvoboda11 (GitHub)
+
+
+OpenBSD driver
+~~~~~~~~~~~~~~
+| Brad Smith
+| brad\@comstyle.com (email), brad (Phabricator), brad0 (GitHub)
+
+
+Driver parts not covered by someone else
+~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
+| Fangrui Song
+| maskray\@google.com (email), MaskRay (Phabricator), MaskRay (GitHub)
+
+
+Tools
+-----
+These code owners are responsible for user-facing tools under the Clang
+umbrella or components used to support such tools.
+
+Tooling library
+~~~~~~~~~~~~~~~
+| Manuel Klimek
+| klimek\@google.com (email), klimek (Phabricator), r4nt (GitHub)
+
+
+clang-format
+~~~~~~~~~~~~
+| MyDeveloperDay
+| mydeveloperday\@gmail.com (email), MyDeveloperDay (Phabricator), MyDeveloperDay (GitHub)
+
+| Owen Pan
+| owenpiano\@gmail.com (email), owenpan (Phabricator), owenca (GitHub)
+
+
+ABIs
+----
+The following people are responsible for decisions involving ABI.
+
+Itanium ABI
+~~~~~~~~~~~
+| John McCall
+| rjmccall\@apple.com (email), rjmccall (Phabricator), rjmccall (GitHub)
+
+
+Microsoft ABI
+~~~~~~~~~~~~~
+| Reid Kleckner
+| rnk\@google.com (email), rnk (Phabricator), rnk (GitHub)
+
+
+ARM EABI
+~~~~~~~~
+| Anton Korobeynikov
+| anton\@korobeynikov.info (email), asl (Phabricator), asl (GitHub)
+
+
+Compiler-Wide Topics
+--------------------
+The following people are responsible for functionality that does not fit into
+a single part of the compiler, but instead span multiple components within the
+compiler.
+
+Attributes
+~~~~~~~~~~
+| Erich Keane
+| erich.keane\@intel.com (email), ErichKeane (Phabricator), erichkeane (GitHub)
+
+
+Inline assembly
+~~~~~~~~~~~~~~~
+| Eric Christopher
+| echristo\@gmail.com (email), echristo (Phabricator), echristo (GitHub)
+
+
+Text encodings
+~~~~~~~~~~~~~~
+| Tom Honermann
+| tom\@honermann.net (email), tahonermann (Phabricator), tahonermann (GitHub)
+
+| Corentin Jabot
+| corentin.jabot\@gmail.com (email), cor3ntin (Phabricator), cor3ntin (GitHub)
+
+
+CMake integration
+~~~~~~~~~~~~~~~~~
+| Petr Hosek
+| phosek\@google.com (email), phosek (Phabricator), petrhosek (GitHub)
+
+| John Ericson
+| git\@johnericson.me (email), Ericson2314 (Phabricator), Ericson2314 (GitHub)
+
+
+General Windows support
+~~~~~~~~~~~~~~~~~~~~~~~
+| Reid Kleckner
+| rnk\@google.com (email), rnk (Phabricator), rnk (GitHub)
+
+
+Standards Conformance
+---------------------
+The following people are responsible for validating that changes are conforming
+to a relevant standard. Contact them for questions about how to interpret a
+standard, when fixing standards bugs, or when implementing a new standard feature.
+
+C conformance
+~~~~~~~~~~~~~
+| Aaron Ballman
+| aaron\@aaronballman.com (email), aaron.ballman (Phabricator), AaronBallman (GitHub)
+
+
+C++ conformance
+~~~~~~~~~~~~~~~
+| Hubert Tong
+| hubert.reinterpretcast\@gmail.com (email), hubert.reinterpretcast (Phabricator), hubert-reinterpretcast (GitHub)
+
+
+Objective-C/C++ conformance
+~~~~~~~~~~~~~~~~~~~~~~~~~~~
+| John McCall
+| rjmccall\@apple.com (email), rjmccall (Phabricator), rjmccall (GitHub)
+
+
+OpenMP conformance
+~~~~~~~~~~~~~~~~~~
+| Alexey Bataev
+| a.bataev\@hotmail.com (email), ABataev (Phabricator), cilkplus (GitHub)
+
+
+OpenCL conformance
+~~~~~~~~~~~~~~~~~~
+| Anastasia Stulova
+| anastasia.stulova\@arm.com (email), Anastasia (Phabricator), AnastasiaStulova (GitHub)
+
+
+SYCL conformance
+~~~~~~~~~~~~~~~~
+| Alexey Bader
+| alexey.bader\@intel.com (email), bader (Phabricator), bader (GitHub)
+
+
+Former Code Owners
+==================
+The following people have graciously spent time performing code ownership
+responsibilities but are no longer active in that role. Thank you for all your
+help with the success of the project!
+
+Emeritus owners
+---------------
+| Doug Gregor (dgregor\@apple.com)
+| Richard Smith (richard\@metafoo.co.uk)
+
+
+Former component owners
+-----------------------
+| Chandler Carruth (chandlerc\@gmail.com, chandlerc\@google.com) -- CMake, library layering
+| Devin Coughlin (dcoughlin\@apple.com) -- Clang static analyzer

diff  --git a/clang/docs/CodeOwners.rst b/clang/docs/CodeOwners.rst
new file mode 100644
index 000000000000..48128fbc5d90
--- /dev/null
+++ b/clang/docs/CodeOwners.rst
@@ -0,0 +1 @@
+.. include:: ../CodeOwners.rst

diff  --git a/clang/docs/index.rst b/clang/docs/index.rst
index 3f392c75dcb5..4406c1aa3e28 100644
--- a/clang/docs/index.rst
+++ b/clang/docs/index.rst
@@ -98,6 +98,7 @@ Design Documents
 .. toctree::
    :maxdepth: 1
 
+   CodeOwners
    InternalsManual
    DriverInternals
    OffloadingDesign

More information about the cfe-commits mailing list