[PATCH] reenable gep merging in some constrainted cases

Quentin Colombet qcolombet at apple.com
Tue May 19 15:00:46 PDT 2015


Hi Wei,

Correct me if I am wrong, but this is a tuning of the heuristic from http://reviews.llvm.org/D8911, right?

Given that you actually see a regression now (and the fact that after discussing with Chandler, I am not convinced the initial change makes sense), I would suggest that you revert r235455 and that we start again the discussion on how to properly fix that.

Thanks,
-Quentin


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9865

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list