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