[llvm] [Matrix] Adjust lifetime.ends during multiply fusion. (PR #84914)

Florian Hahn via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 16 12:44:35 PDT 2024


fhahn wrote:

I evaluated the patch on the codebase where this caused an mis-compile where fusion triggered around 1200 times. In the original version, about 600 lifetime.end markers were removed.

I adjusted the patch to also ignore lifetime.end, if both the loads and store are in the same BB and the lifetime.end is in a different one. In that case, the marker can't be between loads & store. With committed version of the patch, only 3 lifetime.end markers are removed, and 6 are moved.

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


More information about the llvm-commits mailing list