[llvm] [EarlyCSE] De-Duplicate callsites with differing attrs (PR #110929)

[via llvm-commits]

================
@@ -1632,6 +1632,9 @@ bool EarlyCSE::processNode(DomTreeNode *Node) {
           LLVM_DEBUG(dbgs() << "Skipping due to debug counter\n");
           continue;
         }
+
+        // Potential TODO: We may be throwing away attribute information when
+        // we delete Inst that we could propagate too InVal.first.
----------------
goldsteinn wrote:

If we dont combine?

I was thinking about it, are there any **callsite** attributes that can create side-effects?
AFAICT no.

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