Re: r347294 - Use is.constant intrinsic for __builtin_constant_p

2018-11-20 Thread Nico Weber via cfe-commits
I reverted this in r347314 for breaking many bots on lab.llvm.org:8011/console I've also put a standalone repro at https://bugs.chromium.org/p/chromium/issues/detail?id=907099#c2. On Tue, Nov 20, 2018 at 3:56 AM Bill Wendling via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: void >

r347294 - Use is.constant intrinsic for __builtin_constant_p

2018-11-20 Thread Bill Wendling via cfe-commits
Author: void Date: Tue Nov 20 00:53:30 2018 New Revision: 347294 URL: http://llvm.org/viewvc/llvm-project?rev=347294&view=rev Log: Use is.constant intrinsic for __builtin_constant_p Summary: A __builtin_constant_p may end up with a constant after inlining. Use the is.constant intrinsic if it's a