Re: [PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027
On Wed, 2019-08-07 at 17:10 -0300, Adhemerval Zanella wrote: > On 05/06/2019 05:19, Michał Górny via Phabricator via llvm-commits wrote: > > This revision was automatically updated to reflect the committed changes. > > Closed by commit rL362587: [clang] [test] Add a (xfailing) test for PR41027 > > (authored by mgorny, committed by ). > > Herald added a project: LLVM. > > Herald added a subscriber: llvm-commits. > > > > Changed prior to commit: > > https://reviews.llvm.org/D60728?vs=195518=203097#toc > > > > Repository: > > rL LLVM > > > > CHANGES SINCE LAST ACTION > > https://reviews.llvm.org/D60728/new/ > > > > https://reviews.llvm.org/D60728 > > > > Files: > > cfe/trunk/test/Sema/pr41027.c > > This patch is failing on aarch64-linux bot [1], I think you need to add a > 'requires: x86_64'. > > [1] http://lab.llvm.org:8011/builders/clang-cmake-aarch64-full/builds/7694 > Update: I see Bill Wendling already fixed it in r368202. -- Best regards, Michał Górny signature.asc Description: This is a digitally signed message part ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Re: [PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027
On Wed, 2019-08-07 at 17:10 -0300, Adhemerval Zanella wrote: > On 05/06/2019 05:19, Michał Górny via Phabricator via llvm-commits wrote: > > This revision was automatically updated to reflect the committed changes. > > Closed by commit rL362587: [clang] [test] Add a (xfailing) test for PR41027 > > (authored by mgorny, committed by ). > > Herald added a project: LLVM. > > Herald added a subscriber: llvm-commits. > > > > Changed prior to commit: > > https://reviews.llvm.org/D60728?vs=195518=203097#toc > > > > Repository: > > rL LLVM > > > > CHANGES SINCE LAST ACTION > > https://reviews.llvm.org/D60728/new/ > > > > https://reviews.llvm.org/D60728 > > > > Files: > > cfe/trunk/test/Sema/pr41027.c > > This patch is failing on aarch64-linux bot [1], I think you need to add a > 'requires: x86_64'. > > [1] http://lab.llvm.org:8011/builders/clang-cmake-aarch64-full/builds/7694 > Thanks for the report. You're probably right. I'll verify the correct REQUIRES for it, and commit it in a few minutes. -- Best regards, Michał Górny signature.asc Description: This is a digitally signed message part ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Re: [PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027
On 05/06/2019 05:19, Michał Górny via Phabricator via llvm-commits wrote:
> This revision was automatically updated to reflect the committed changes.
> Closed by commit rL362587: [clang] [test] Add a (xfailing) test for PR41027
> (authored by mgorny, committed by ).
> Herald added a project: LLVM.
> Herald added a subscriber: llvm-commits.
>
> Changed prior to commit:
> https://reviews.llvm.org/D60728?vs=195518=203097#toc
>
> Repository:
> rL LLVM
>
> CHANGES SINCE LAST ACTION
> https://reviews.llvm.org/D60728/new/
>
> https://reviews.llvm.org/D60728
>
> Files:
> cfe/trunk/test/Sema/pr41027.c
This patch is failing on aarch64-linux bot [1], I think you need to add a
'requires: x86_64'.
[1] http://lab.llvm.org:8011/builders/clang-cmake-aarch64-full/builds/7694
>
>
> Index: cfe/trunk/test/Sema/pr41027.c
> ===
> --- cfe/trunk/test/Sema/pr41027.c
> +++ cfe/trunk/test/Sema/pr41027.c
> @@ -0,0 +1,10 @@
> +// RUN: %clang_cc1 -triple x86_64 -fsyntax-only %s
> +// XFAIL: *
> +
> +inline void pr41027(unsigned a, unsigned b) {
> + if (__builtin_constant_p(a)) {
> +__asm__ volatile("outl %0,%w1" : : "a"(b), "n"(a));
> + } else {
> +__asm__ volatile("outl %0,%w1" : : "a"(b), "d"(a));
> + }
> +}
>
>
>
> ___
> llvm-commits mailing list
> llvm-comm...@lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Re: [PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027
On Wed, Apr 17, 2019 at 05:34:04PM +, Kamil Rytarowski via Phabricator wrote: > Right now this blocks upgrades on NetBSD as a number of CPUs is affected (but > not x86, so it was overlooked before 8.0 by others than @joerg). Note that this is not about anything really specific to NetBSD, the kind of conditional asm statements are pretty popular. Joerg ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
