[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