tree c72522764e427f9f9fbe32f4f8e711bf7930a4fc
parent b812fd634ea5ff307c374fb2b6340169f7db8b16
author Jaebaek Seo <jaebaek@google.com> 1611856655 -0500
committer GitHub <noreply@github.com> 1611856655 -0500
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJgEvsPCRBK7hj4Ov3rIwAAdHIIAJ5zPYZzmnmPqDkGomZbQNEh
 dGeHpHypiZPar44YOmPHpSOaGPa9Uhxql4CqqThKJ6cg0X4uOCCBpKfIdN7JSsjh
 iEW5F4ZgnVpCUDfizdbsx9jXMvmfOV7W20dh4hVeOoPN4VftS5Tn7rVKopBlQyWC
 5cvJMl8ApqAX8Z9h3y9gUNKhom2+8wEFSBp+KmqWDDLs5i9tR8xNyf9hKe8UL2/U
 EMQmeu7Bd5zZK1fWtw7ANFjPihLidXKZQcURnkUC5WSr3bPhF9QbH5Ak15JDAVAK
 z76HXYYKaCzzb719vgJNxAIF9a3GBWNb/QB+r9Y0iiM3QscRS50pza/L6d1vi78=
 =JEmU
 -----END PGP SIGNATURE-----
 

Set correct scope and line info for DebugValue (#4125)

The existing spirv-opt `DebugInfoManager::AddDebugValueForDecl()` sets
the scope and line info of the new added DebugValue using the scope and
line of DebugDeclare. This is wrong because only a single DebugDeclare
must exist under a scope while we have to add DebugValue for all the
places where the variable's value is updated. Therefore, we have to set
the scope and line of DebugValue based on the places of the variable
updates.

This bug makes
https://github.com/google/amber/blob/main/tests/cases/debugger_hlsl_shadowed_vars.amber
fail. This commit fixes the bug.