Code review comment for lp:~christophe-lyon/gcc-linaro/4.7-backport-vect-cost-model

Revision history for this message
Yvan Roux (yvan-roux) wrote :

* Diff matches upstream, but Changelog.linaro doesn't include the removing of the unrelevant 4.8 code:
  - removing of arm_add_stmt_cost, TARGET_VECTORIZE_ADD_STMT_COST and the handling of the enum vect_cost_for_stmt value vec_construct in arm_builtin_vectorization_cost.

* testsuite is fine (with the usual qemu threading issues on cortexa9hf and cortexa15hf)
* benchmarks are ok

« Back to merge proposal