jansvoboda11 marked an inline comment as done.
jansvoboda11 added a comment.

In D134653#3815672 <https://reviews.llvm.org/D134653#3815672>, @srj wrote:

> FYI, adding this patch to my local build does allow me to *compile* properly 
> now. I haven't tested for correctness.

Thanks for verifying the build. You can test for correctness by "building" the 
`check-clang` target.

In D134653#3815648 <https://reviews.llvm.org/D134653#3815648>, @srj wrote:

> Regarding presubmit testing, that's of great concern to me; if LLVM isn't 
> testing on x86-32 then it's not really *supporting* x86-32 anymore (as a 
> compilation host anyway), and that's a rather large change to make without 
> warning. Do you know to whom I should escalate that concern?

You can contact the infrastructure working group. Note that we run more 
configurations post-commit: https://lab.llvm.org/buildbot/#/console


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134653

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

Reply via email to