[llvm] [NVPTX] Disable incorrect peephole optimizations (PR #79920)

Justin Fargnoli via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 8 13:35:15 PST 2024


justinfargnoli wrote:

TLDR: I won't have the bandwidth to fix this in the foreseeable future. 

I implemented the fix that you suggested (and I agree that it's likely the right way to go). However, it wasn't trivial to fix the unit tests. That gave me pause about submitting a fix that I don't fully understand. e.g. I haven't run tests to assess the performance impact of the fix and I don't understand the original rationale for the change. So, instead of doing a deep dive on the fix, I'd like to remove the erroneous code for the time being and revisit it when necessary.  

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


More information about the llvm-commits mailing list