Re: r348070 - Specify constant context in constant emitter

2018-12-01 Thread Richard Smith via cfe-commits
This seems is wrong way to handle this, and seems likely to be unsound in C++. We should be creating a ConstantExpr node wrapped around the initializer instead. On Sat, 1 Dec 2018, 00:32 Bill Wendling via cfe-commits < cfe-commits@lists.llvm.org wrote: > Author: void > Date: Sat Dec 1 00:29:36 2

r348070 - Specify constant context in constant emitter

2018-12-01 Thread Bill Wendling via cfe-commits
Author: void Date: Sat Dec 1 00:29:36 2018 New Revision: 348070 URL: http://llvm.org/viewvc/llvm-project?rev=348070&view=rev Log: Specify constant context in constant emitter The constant emitter may need to evaluate the expression in a constant context. For exasmple, global initializer lists.