[all-commits] [llvm/llvm-project] 222198: [clang][Interp] Add scopes to conditional operator...

[Timm Baeder via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 22219873987587d8b5d793ab5dea982a0887ac7c
      https://github.com/llvm/llvm-project/commit/22219873987587d8b5d793ab5dea982a0887ac7c
  Author: Timm Baeder <tbaeder at redhat.com>
  Date:   2024-08-16 (Fri, 16 Aug 2024)

  Changed paths:
    M clang/lib/AST/Interp/Compiler.cpp
    M clang/lib/AST/Interp/Compiler.h
    M clang/test/AST/Interp/records.cpp

  Log Message:
  -----------
  [clang][Interp] Add scopes to conditional operator subexpressions (#104418)

We would otherwise try to destroy the variables from both branches after
the conditional operator was done.

However, doing so breaks the case where we create internal temporaries.
For those, add allocateTemporary(), which attaches the lifetime of the
temporary to the topmost scope. Since they never have record type, this
shouldn't create any issues.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications