On Thu, 2016-12-01 at 16:05 -0700, Jeff Law wrote: > On 11/11/2016 02:15 PM, David Malcolm wrote: > > Posted earlier here: > > https://gcc.gnu.org/ml/gcc-patches/2016-11/msg00651.html > > Link to earlier discussion: > > https://gcc.gnu.org/ml/gcc-patches/2016-10/msg01801.html > > > > This version: > > - eliminates the rtx_reuse_manager singleton > > - eliminates print-rtl-reuse.h, moving class rtx_reuse_manager into > > print-rtl.h and print-rtl.c > > - fixes the formatting issues you noted in the earlier patch > Can you repost just those pieces that still need review? I know > various > bits have been approved and various bits have iterated. Just want to > see what's really still outstanding.
Here's the current status of the kit: "[PATCH 1/9] print_rtx: implement support for reuse IDs (v2)" https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01188.html Still needs review Patches 2-4 are approved. "[PATCH 5/9] Introduce selftest::locate_file (v4)" https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01186.html Mostly approved, but some discussion ongoing "[PATCH 6/9] Split class rtx_reader into md_reader vs rtx_reader" https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01184.html Approved (by Richard Sandiford) "[PATCH 7/9] Add RTL-error-handling to host" https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01181.html Bernd and I are still discussing how best to do this. "[PATCH 8/9] Introduce class function_reader (v4)" https://gcc.gnu.org/ml/gcc-patches/2016-11/msg01185.html Bernd and I are discussing this; this has now split into: * [PATCH 8a/9] Introduce class function_reader (v6) * https://gcc.gnu.org/ml/gcc-patches/2016-12/msg00149.html * [PATCH 8b/9] Add target-independent selftests of RTL function reader * https://gcc.gnu.org/ml/gcc-patches/2016-12/msg00147.html * [PATCH 8c/9] Add aarch64-specific selftests for RTL function reader * https://gcc.gnu.org/ml/gcc-patches/2016-12/msg00148.html * [PATCH 8d/9] Add x86_64-specific selftests for RTL function reader * https://gcc.gnu.org/ml/gcc-patches/2016-12/msg00150.html "[PATCH 9/9] Add "__RTL" to cc1 (v6)" https://gcc.gnu.org/ml/gcc-patches/2016-12/msg00151.html Needs review. So patch 1 plus patches 7-9 are still outstanding. Hope this answers your question. Dave