erichkeane added a comment.

In D142388#4099374 <https://reviews.llvm.org/D142388#4099374>, @ManuelJBrito 
wrote:

> So this is getting some build failures:
>
> - https://lab.llvm.org/buildbot#builders/38/builds/9446
> - https://lab.llvm.org/buildbot#builders/245/builds/4189
> - https://lab.llvm.org/buildbot#builders/65/builds/7949
> - https://lab.llvm.org/buildbot#builders/188/builds/25538
>
> I suspect it's because of the way different ABIs handle the vectors.  I was 
> thinking that if we have the variable be a local variable and also not 
> returning the value can fix the tests.
>
> Does this make sense?

Yep, thats what it looks like, ABI differences.  Feel free to update this 
review with the new test now that you've been reverted, and I'll look to see if 
it seems sufficient.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142388/new/

https://reviews.llvm.org/D142388

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to