[llvm] [DSE] Stop double-counting the dead slice offset in shortenAssignment (PR #216227)

Eric Christopher via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 16 23:27:08 PDT 2026


echristo wrote:

> I feel I'm being a bit pedantic with my nits, sorry!

Not a single problem at all. Good comments are important.

> 
> Just out of curiosity, because you mention the patch is assisted by AI, does that extend to the comments in the tests, or are these all human-written?

A little bit of both honestly. I've always felt like I've under-commented my testcases (10 years later - "what on earth did that mean?") and so perhaps went a little overboard with some help. Thanks for checking in and for the comment review. It's appreciated. I've gone ahead and updated that first section - see if that's a bit better explanation of the what the test case is designed to test :)

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


More information about the llvm-commits mailing list