[all-commits] [llvm/llvm-project] 910292: [LLVM][Coroutines] Check variable decl scope inste...

[Billy Zhu via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 910292c3ac2ebe43cdbc90223c6c9702128316db
      https://github.com/llvm/llvm-project/commit/910292c3ac2ebe43cdbc90223c6c9702128316db
  Author: Billy Zhu <billyzhu at modular.com>
  Date:   2024-05-23 (Thu, 23 May 2024)

  Changed paths:
    M llvm/lib/Transforms/Coroutines/CoroFrame.cpp
    R llvm/test/Transforms/Coroutines/coro-debug-frame-variable-O1.ll
    A llvm/test/Transforms/Coroutines/coro-debug-frame-variable-inlined.ll

  Log Message:
  -----------
  [LLVM][Coroutines] Check variable decl scope instead of optimization level for hoisted DbgDeclare Loc (#92978)

Minor patch following up on
https://github.com/llvm/llvm-project/pull/75402.

The more generalized version of [this
error](https://github.com/llvm/llvm-project/pull/75104#issuecomment-1853497609)
is whenever we have a debug variable created in one subprogram scope
inlined into another subprogram scope. So instead of checking
optimization level, it is safer to just check whether the subprogram
scope of the variable matches the subprogram scope of the hoisted
position.



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