[PATCH] [X86] Memory folding for commutative instructions (updated)

Quentin Colombet qcolombet at apple.com
Fri Oct 17 12:24:11 PDT 2014


Hi Simon,

Thanks for the update.
Few things I missed from the previous review on the regression test:
- Please do not put the date in the file name.
- Add some check lines (and add FileCheck on the run line) to be sure we generate reasonable code were it used to crash.

Sorry for missing that the first time.

Thanks,
-Quentin

http://reviews.llvm.org/D5818






More information about the llvm-commits mailing list