[llvm] [SDAG] Read-only intrinsics must have WillReturn attribute to be treated as loads (PR #99999)

[Kevin McAfee via llvm-commits]

kalxr wrote:

> So after #98968, you no longer have a case that actually hits this situation?
I think the bpf.load intrinsics are in a similar state to the nvvm.ldu/ldg intrinsics, which might be why the bpf test change is needed. Those should probably have `IntrWillReturn`. Other than that, I'm not aware of any public intrinsic that hits this case. Even if one existed, it would need to have a workaround in place to avoid getting removed (like using some other attribute to signal side effects) so I'm not sure how to find it.



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