Re: r345591 - [CodeGen] Disable the machine verifier on a ThinLTO test

2018-11-06 Thread Francis Visoiu Mistrih via cfe-commits
Thanks for the suggestion, I think it’s reasonable since it’s all due to: * ICALL_BRANCH_FUNNEL: Bad machine code: Explicit definition marked as use https://bugs.llvm.org/show_bug.cgi?id=39436 . I’ll look into it. > On 5 Nov 2018, at 19:13, David Bl

Re: r345591 - [CodeGen] Disable the machine verifier on a ThinLTO test

2018-11-05 Thread David Blaikie via cfe-commits
If ThinLTO doesn't pass the machine verifier - should it maybe be turned off at the thinlto level in general, rather than for this specific test? On Tue, Oct 30, 2018 at 5:20 AM Francis Visoiu Mistrih via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: thegameg > Date: Tue Oct 30 05:18

r345591 - [CodeGen] Disable the machine verifier on a ThinLTO test

2018-10-30 Thread Francis Visoiu Mistrih via cfe-commits
Author: thegameg Date: Tue Oct 30 05:18:33 2018 New Revision: 345591 URL: http://llvm.org/viewvc/llvm-project?rev=345591&view=rev Log: [CodeGen] Disable the machine verifier on a ThinLTO test This allows us to turn the machine verifier on by default on X86. Modified: cfe/trunk/test/CodeGen/t