[PATCH] D42032: [LLVM][PASSES][InstCombine] Fix (a + a + ...) / a cases

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jan 14 09:00:27 PST 2018


lebedev.ri added a comment.

1. Please upload patches with full context (`-U999999`)
2. This needs testing coverage https://bugs.llvm.org/show_bug.cgi?id=35709#c1 notes that "(x+x+x)/x" is already handled, so maybe search for it and look how it is tested? And given that this is a generalization of that, perhaps that one is no longer needed?


Repository:
  rL LLVM

https://reviews.llvm.org/D42032





More information about the llvm-commits mailing list