Paul C. Anagnostopoulos via llvm-dev
2020-Sep-22 18:33 UTC
[llvm-dev] Now I really have broken the build
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.
Hubert Tong via llvm-dev
2020-Sep-22 18:40 UTC
[llvm-dev] Now I really have broken the build
On Tue, Sep 22, 2020 at 2:33 PM Paul C. Anagnostopoulos via llvm-dev < llvm-dev at lists.llvm.org> wrote:> 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;> > _______________________________________________ > LLVM Developers mailing list > llvm-dev at lists.llvm.org > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20200922/4a97d603/attachment.html>
Arthur Eubanks via llvm-dev
2020-Sep-22 18:47 UTC
[llvm-dev] Now I really have broken the build
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. On Tue, Sep 22, 2020 at 11:41 AM Hubert Tong via llvm-dev < llvm-dev at lists.llvm.org> wrote:> On Tue, Sep 22, 2020 at 2:33 PM Paul C. Anagnostopoulos via llvm-dev < > llvm-dev at lists.llvm.org> wrote: > >> 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; > > >> >> _______________________________________________ >> LLVM Developers mailing list >> llvm-dev at lists.llvm.org >> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev >> > _______________________________________________ > LLVM Developers mailing list > llvm-dev at lists.llvm.org > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev >-------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20200922/da6b721a/attachment.html>