------- Comment #26 from bernds at gcc dot gnu dot org 2010-08-19 13:38 ------- (In reply to comment #25) > Alex Oliva posted some patches to make cselib handle autoinc stuff. > No idea whether http://gcc.gnu.org/ml/gcc-patches/2010-03/msg01038.html > is the latest version or if he has a newer one.
Doesn't look bad, but I dislike the name change to _addr variants for functions that look up not just addresses. I think we should just fix up all the callers to pass in a mode. I'm also not sure we need a global cselib_record_autoinc flag - why not just do that always? Shouldn't we do something else when hashing PRE_MODIFY? -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43494