Now I really have broken the build

Project Infrastructure LLVM Dev List Archives
#1

There is a second issue with the new TableGenBackendSkeleton.cpp file. It is a skeleton file for writing a new TableGen backend.

/home/buildbots/docker-RHEL-buildbot/SetupBot/worker_env/ppc64le-clang-rhel-test/clang-ppc64le-rhel/llvm/llvm/lib/TableGen/TableGenBackendSkeleton.cpp:38:17: error: private field 'Records' is not used [-Werror,-Wunused-private-field]
  RecordKeeper &Records;
                ^
There are two options:

1. Don't compile it at all. (I swear I compiled it locally.)

2. Add something to eliminate the error.

I'm open to suggestions.

#2

There is a second issue with the new TableGenBackendSkeleton.cpp file. It is a skeleton file for writing a new TableGen backend.

/home/buildbots/docker-RHEL-buildbot/SetupBot/worker_env/ppc64le-clang-rhel-test/clang-ppc64le-rhel/llvm/llvm/lib/TableGen/TableGenBackendSkeleton.cpp:38:17: error: private field ‘Records’ is not used [-Werror,-Wunused-private-field]
RecordKeeper &Records;
^
There are two options:

  1. Don’t compile it at all. (I swear I compiled it locally.)

  2. Add something to eliminate the error.

I’m open to suggestions.

The run function would normally reference the field, no?

So:
// Something that references the “Records” field.
(void)Records;

#3

As a general note, you can always revert the commit, then fix it and recommit, that way head isn’t broken while investigating.

For the Sphinx issue, there needs to be a reference to the new page somewhere, there should be a page somewhere with links to (almost) everything where you can add the new page.