[PATCH] D40049: [PATCH] Global reassociation for improved CSE

[Sanjay Patel via Phabricator via llvm-commits]

spatel added a comment.

My understanding of 'reassociate' isn't good enough to approve this, but I'm curious about running the main loop multiple times (ReassociateStep) because I noticed an improvement in some other code by running the pass twice.

1. Can we split that part into a preliminary patch?
2. Is it too expensive in compile time to run that to a fixed-point (like instcombine)?


Repository:
  rL LLVM

https://reviews.llvm.org/D40049