[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 did CSE for non-readonly funtions ofc I could see param args mattering, but with the limits I think its only return?

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