Re: [PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-08-07 Thread Michał Górny via cfe-commits
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 > >

Re: [PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-08-07 Thread Michał Górny via cfe-commits
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 > >

Re: [PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-08-07 Thread Adhemerval Zanella via cfe-commits
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.

[PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-06-05 Thread Michał Górny via Phabricator via cfe-commits
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://revi

Re: [PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-04-17 Thread Joerg Sonnenberger via cfe-commits
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

[PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-04-17 Thread Kamil Rytarowski via Phabricator via cfe-commits
krytarowski added a comment. In D60728#1469794 , @hans wrote: > In D60728#1468713 , @krytarowski > wrote: > > > In D60728#1468486 , @hans wrote: > > > > > What's the value i

[PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-04-17 Thread Michał Górny via Phabricator via cfe-commits
mgorny updated this revision to Diff 195518. mgorny added a comment. Updated per request. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60728/new/ https://reviews.llvm.org/D60728 Files: clang/test/Sema/pr41027.c Index: clang/test/Sema/pr41027.c =

[PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-04-17 Thread Hans Wennborg via Phabricator via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. In D60728#1468713 , @krytarowski wrote: > In D60728#1468486 , @hans wrote: > > > What's the value in checking in th

[PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-04-16 Thread Michał Górny via Phabricator via cfe-commits
mgorny added a comment. In D60728#1468486 , @hans wrote: > What's the value in checking in this xfail'ed test without an actual fix for > the problem? 1. It may help whoever tries to address it in the future, to have a known-good reproducer. 2. If so

[PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-04-16 Thread Kamil Rytarowski via Phabricator via cfe-commits
krytarowski added a comment. In D60728#1468486 , @hans wrote: > What's the value in checking in this xfail'ed test without an actual fix for > the problem? Raise awareness about the problem. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60728/

[PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-04-16 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. What's the value in checking in this xfail'ed test without an actual fix for the problem? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60728/new/ https://reviews.llvm.org/D60728 ___ cfe-commits mailing list cfe-commi

[PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-04-16 Thread Kamil Rytarowski via Phabricator via cfe-commits
krytarowski added a reviewer: void. krytarowski added a subscriber: void. krytarowski added a comment. Adding @void, regression introduced with D55616 . CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60728/new/ https://reviews.llvm.org/D60728

[PATCH] D60728: [clang] [test] Add a (xfailing) test for PR41027

2019-04-15 Thread Michał Górny via Phabricator via cfe-commits
mgorny created this revision. mgorny added reviewers: krytarowski, joerg, hans, rsmith. Add a test for tracking PR41027 (8.0 regression breaking assembly code relying on __builtin_constant_p() to identify compile-time constants). Mark it as expected to fail everywhere. https://reviews.llvm.org/D