[PATCH] D39424: [Reassociate] Remove FIXME from looptest.ll (NFC)

Chad Rosier via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 30 10:26:11 PDT 2017


mcrosier accepted this revision.
mcrosier added a comment.

This probably comes from when I was adding support for FP in the reassociation pass many years ago.  I don't recall exactly why I added the FIXME, but the original test was checking that (i+j) gets reassociated, so I agree with Florian's change.


https://reviews.llvm.org/D39424





More information about the llvm-commits mailing list