[llvm] [DSE] Refactor DSE (PR #100956)

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 8 08:52:32 PDT 2024


aeubanks wrote:

> > I thought the code was originally in DSEState?
> 
> The major code was originally in `static bool eliminateDeadStores(...)`, this loop: https://github.com/llvm/llvm-project/blob/main/llvm/lib/Transforms/Scalar/DeadStoreElimination.cpp#L2145

Moving a function from a static function to a DSEState method seems fine, I don't see anything wrong with that

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


More information about the llvm-commits mailing list