Chad Rosier b81558a8e1 [Reassociate] Add negated value of negative constant to the Duplicates list. ... In OptimizeAdd, we scan the operand list to see if there are any common factors between operands that can be factored out to reduce the number of multiplies (e.g., 'A*A+A*B*C+D' -> 'A*(A+B*C)+D'). For each operand of the operand list, we only consider unique factors (which is tracked by the Duplicate set). Now if we find a factor that is a negative constant, we add the negated value as a factor as well, because we can percolate the negate out. However, we mistakenly don't add this negated constant to the Duplicates set. Consider the expression A*2*-2 + B. Obviously, nothing to factor. For the added value A*2*-2 we over count 2 as a factor without this change, which causes the assert reported in PR30256. The problem is that this code is assuming that all the multiply operands of the add are already reassociated. This change avoids the issue by making OptimizeAdd tolerate multiplies which haven't been completely optimized; this sort of works, but we're doing wasted work: we'll end up revisiting the add later anyway. Another possible approach would be to enforce RPO iteration order more strongly. If we have RedoInsts, we process them immediately in RPO order, rather than waiting until we've finished processing the whole function. Intuitively, it seems like the natural approach: reassociation works on expression trees, so the optimization only works in one direction. That said, I'm not sure how practical that is given the current Reassociate; the "optimal" form for an expression depends on its use list (see all the uses of "user_back()"), so Reassociate is really an iterative optimization of sorts, so any changes here would probably get messy. PR30256 Differential Revision: https://reviews.llvm.org/D30228 git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@296003 91177308-0d34-0410-b5e6-96231b3b80d8		2017-02-23 18:49:03 +00:00
..
Coroutines	[Coroutines] Add header guard to header that's missing one.	2017-01-30 16:32:20 +00:00
Hello	Add auto-exporting of symbols from tools so that plugins work on Windows	2016-05-26 11:16:43 +00:00
InstCombine	[InstCombine] use loop instead of recursion to peek through FPExt; NFCI	2017-02-23 16:39:51 +00:00
Instrumentation	[AddressSanitizer] Add PS4 offset	2017-02-23 17:10:28 +00:00
IPO	Use base discriminator in sample pgo profile matching.	2017-02-23 18:27:45 +00:00
ObjCARC	[CMake] NFC. Updating CMake dependency specifications	2016-11-17 04:36:50 +00:00
Scalar	[Reassociate] Add negated value of negative constant to the Duplicates list.	2017-02-23 18:49:03 +00:00
Utils	PredicateInfo: Support switch statements	2017-02-22 22:20:58 +00:00
Vectorize	[SLP] Fix for PR32036: Vectorized horizontal reduction returning wrong	2017-02-23 13:37:09 +00:00
CMakeLists.txt	[coroutines] Part 3 of N: Adding Boilerplate for Coroutine Passes	2016-07-28 21:04:31 +00:00
LLVMBuild.txt	[coroutines] Part 3 of N: Adding Boilerplate for Coroutine Passes	2016-07-28 21:04:31 +00:00