[llvm] [SDAG] Use useDebugInstrRef instead of shouldUseDebugInstrRef (PR #160686)

[Evgenii Kudriashov via llvm-commits]

e-kud wrote:

> > I think it is worth mentioning that the main problem happens while using opt-bisect. When the optimization level changes. My concern here is whether we need `shouldUseDebugInstrRef` at all? Or maybe we need to check whether we are bisecting in `shouldUseDebugInstrRef` and return a consistent result for it?
> > UPD: We need some testing as well...
> 
> My understanding of the `shouldUseDebugInstrRef` is that it's a helper that's used to set `UseInstrRef`; it should never be used outside of initialization (and indeed the usage removed in this PR is the only one)

Oh, I see. I just saw three uses, and they looked very similar to me. But indeed this one should use the attribute that we've just set earlier. 

https://github.com/llvm/llvm-project/pull/160686