[llvm] [LoopIdiomRecognizer] Implement CRC recognition (PR #79295)
Mikhail Gudim via llvm-commits
llvm-commits at lists.llvm.org
Wed Jun 19 13:17:55 PDT 2024
mgudim wrote:
@joe-img I went through the code and just from looking this seems to work correctly. However, I still stand by my suggestions in this comment: https://github.com/llvm/llvm-project/pull/79295#issuecomment-1962269129
I agree that the interpreter approach is better than trying to recognize all variations of the pattern in the source code.
I don't insist that you follow my suggestions and I am OK with someone else approving this as long as this is not enabled by default. This way we can merge your work and maybe improve on it gradually.
https://github.com/llvm/llvm-project/pull/79295
More information about the llvm-commits
mailing list