<div dir="ltr">That looks correct, but it is very strange to me that it would be needed as the result of my original patch, which just included AddressSpace.hpp from a new test.<div><br></div><div>But how libunwind configuration works is a big mystery to me.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 13, 2020 at 1:40 AM Mikael Holmén <<a href="mailto:mikael.holmen@ericsson.com">mikael.holmen@ericsson.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi,<br>
<br>
I still got the warnings with this fix so I pushed something in<br>
abc238a3c7361 that at least makes the compiler quiet when i build.<br>
<br>
Please take a look and see if that's reasonable or if that fix should<br>
be reverted.<br>
<br>
/Mikael<br>
<br>
On Thu, 2020-03-12 at 18:18 -0700, Sterling Augustine via llvm-commits<br>
wrote:<br>
> Should be fixed by 3758b858ef3b39b3081bf9a567ead8bc69c9a208.<br>
> <br>
> But because I can't test the build, I can't verify.<br>
> <br>
> On Thu, Mar 12, 2020 at 5:42 PM Sterling Augustine <<br>
> <a href="mailto:saugustine@google.com" target="_blank">saugustine@google.com</a>> wrote:<br>
> > I will fix.<br>
> > <br>
> > On Thu, Mar 12, 2020 at 3:45 PM Vlad Vereschaka via Phabricator <<br>
> > <a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>> wrote:<br>
> > > vvereschaka added a comment.<br>
> > > <br>
> > > Hello @saugustine,<br>
> > > <br>
> > > here is failed libunwind test on Aarch64 toolchain builder<br>
> > > <br>
<a href="http://lab.llvm.org:8011/builders/llvm-clang-win-x-aarch64/builds/5660" rel="noreferrer" target="_blank">http://lab.llvm.org:8011/builders/llvm-clang-win-x-aarch64/builds/5660</a><br>
> > > <br>
> > > - FAIL: libunwind:: frameheadercache_test.pass.cpp<br>
> > > <br>
> > > with a set of similar errors:<br>
> > > <br>
> > >   C:\buildbot\as-builder-2\llvm-clang-win-x-aarch64\llvm-<br>
> > > project\libunwind\test/../src/Registers.hpp:3158:59: error:<br>
> > > unused parameter 'regNum' [-Werror,-Wunused-parameter]<br>
> > > <br>
> > >   inline bool Registers_mips_newabi::validFloatRegister(int<br>
> > > regNum) const {<br>
> > > <br>
> > > would you take care of it?<br>
> > > <br>
> > > <br>
> > > Repository:<br>
> > >   rG LLVM Github Monorepo<br>
> > > <br>
> > > CHANGES SINCE LAST ACTION<br>
> > >   <a href="https://reviews.llvm.org/D75954/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D75954/new/</a><br>
> > > <br>
> > > <a href="https://reviews.llvm.org/D75954" rel="noreferrer" target="_blank">https://reviews.llvm.org/D75954</a><br>
> > > <br>
> > > <br>
> > > <br>
> <br>
> _______________________________________________<br>
> llvm-commits mailing list<br>
> <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br>
> <br>
<a href="https://protect2.fireeye.com/v1/url?k=f22a25f0-aea0f0e6-f22a656b-862f14a9365e-3d839ca728c3c5be&q=1&e=eb012d65-544d-40b3-a81e-577ee1b91493&u=https%3A%2F%2Flists.llvm.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Fllvm-commits" rel="noreferrer" target="_blank">https://protect2.fireeye.com/v1/url?k=f22a25f0-aea0f0e6-f22a656b-862f14a9365e-3d839ca728c3c5be&q=1&e=eb012d65-544d-40b3-a81e-577ee1b91493&u=https%3A%2F%2Flists.llvm.org%2Fcgi-bin%2Fmailman%2Flistinfo%2Fllvm-commits</a><br>
</blockquote></div>