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

[escha via Phabricator via llvm-commits]

escha updated this revision to Diff 125201.
escha added a comment.

Updated patch and added comments.

My main change is that it no longer runs reassociate twice; it turns out despite my intuition this helps EXTREMELY little on my shader test suite! It looks like it's better to just build the pair map at the start even if it's not 100% accurate.


Repository:
  rL LLVM

https://reviews.llvm.org/D40049

Files:
  include/llvm/Transforms/Scalar/Reassociate.h
  lib/Transforms/Scalar/Reassociate.cpp
  test/Transforms/Reassociate/basictest.ll
  test/Transforms/Reassociate/fast-ReassociateVector.ll
  test/Transforms/Reassociate/fast-fp-commute.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D40049.125201.patch
Type: text/x-patch
Size: 8773 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171201/a03d20b4/attachment.bin>