On Sat, Feb 10, 2018 at 5:29 AM, Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> wrote: > During simplify_one_symbol(), if possible, loads are replaced by > an OP_PHI and the corresponding OP_PHISOURCE. To simplify things > firther, If all the phisrcs correspond to an unique pseudo (often > because there is only a single phisrc), then it's useless to > create the OP_PHI: the created OP_PHISOURCEs can be removed and > the initial load can be converted to the unique pseudo. > > However, if the unique pseudo was never used, the removal of > the OP_PHISOURCEs, done *before* the load conversion, will > kill the defining load (at this point the only user of the > pseudo was the OP_PHISOURCEs) which will then erroneously make > a VOID from the pseudo. > > Fix this by doing the load conversion before removing the > unneeded OP_PHISOURCEs. That make sense. Thanks Chris -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
