[PATCH] [GC] Turn {undef,null} transforms into strategy

Philip Reames listmail at philipreames.com
Sun Jan 4 15:25:29 PST 2015


What are you trying to accomplish here?  The patch appears slightly nonsensical.  The lowering code your modifying is from the old gc.root mechanism and isn't involved with statepoints.

Why do you want to move the optimization code out of InstCombine?  Arguably some of it should live in InstSimplify, but it's definitely an IR level optimization.

Do you have a collector which doesn't want these?


http://reviews.llvm.org/D6837

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






More information about the llvm-commits mailing list