[clang] [clang][bytecode] Improve __builtin_{,dynamic_}object_size implementation (PR #153601)

[Shafik Yaghmour via cfe-commits]

================
@@ -60,16 +60,18 @@ template <class Emitter> class OptionScope final {
 public:
   /// Root constructor, compiling or discarding primitives.
   OptionScope(Compiler<Emitter> *Ctx, bool NewDiscardResult,
-              bool NewInitializing)
+              bool NewInitializing, bool NewToLValue)
       : Ctx(Ctx), OldDiscardResult(Ctx->DiscardResult),
-        OldInitializing(Ctx->Initializing) {
+        OldInitializing(Ctx->Initializing), OldToLValue(NewToLValue) {
----------------
shafik wrote:

Should you be using `Ctx->ToLValue` to initialize `OldToLValue`?

https://github.com/llvm/llvm-project/pull/153601
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits