Re: r253595 - Update clang tests to accomendate discriminator changes.

2015-11-19 Thread David Blaikie via cfe-commits
On Thu, Nov 19, 2015 at 12:42 PM, Dehao Chen wrote: > Yes, as discriminator is a backend pass, it should *not* affect frontend > tests. > Could you update these tests to not involve discriminators by passing -disable-llvm-optzns instead? > > On Thu, Nov 19, 2015 at 12:36 PM, David Blaikie > w

Re: r253595 - Update clang tests to accomendate discriminator changes.

2015-11-19 Thread Dehao Chen via cfe-commits
Yes, as discriminator is a backend pass, it should *not* affect frontend tests. On Thu, Nov 19, 2015 at 12:36 PM, David Blaikie wrote: > > > On Thu, Nov 19, 2015 at 11:53 AM, Dehao Chen via cfe-commits > wrote: >> >> Author: dehao >> Date: Thu Nov 19 13:53:16 2015 >> New Revision: 253595 >> >> U

Re: r253595 - Update clang tests to accomendate discriminator changes.

2015-11-19 Thread David Blaikie via cfe-commits
On Thu, Nov 19, 2015 at 11:53 AM, Dehao Chen via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: dehao > Date: Thu Nov 19 13:53:16 2015 > New Revision: 253595 > > URL: http://llvm.org/viewvc/llvm-project?rev=253595&view=rev > Log: > Update clang tests to accomendate discriminator change

r253595 - Update clang tests to accomendate discriminator changes.

2015-11-19 Thread Dehao Chen via cfe-commits
Author: dehao Date: Thu Nov 19 13:53:16 2015 New Revision: 253595 URL: http://llvm.org/viewvc/llvm-project?rev=253595&view=rev Log: Update clang tests to accomendate discriminator changes. Summary: The discriminator change in http://reviews.llvm.org/D14738 will fail these clang tests. Update the