On Tue, Dec 03, 2019 at 12:57:24PM -0500, Michael Meissner wrote: > No, the change for num_insns_constant_gpr could not go in until the support > for > PLI went in (patch V6 #1).
Well, I lost track. So your version 7 to 9 patches do *not* replace the v6 patches? Or does "V" mean something else? Please post patches you propose currently (after addressing comments, saying what is changed wrt the previous submission, etc.) And if you restructure the patch sets, e.g. add new things to patches, I will have to start reviewing from scratch. While if you just do some modifications (and you say what you modified, and that matches reality), and those modifications are in line with previous reviews, then reviewing makes easy progress. It also normally would not take a single day even to make such changes. > So for patches V6 #1-3, I believe you approved #1 (movdi) and #3 > (addi3/addsi3) > after the changes but not #2 (movsi). I haven't yet applied the specific bits > of #1 or #3 after doing the reformating patch. Are the bits for #2 ok with > the > above change to use num_insns_constant_gpr. Or did you want to see the 3 > patches once again after the reformating. https://gcc.gnu.org/ml/gcc-patches/2019-10/msg01605.html This is six weeks ago. And I do not know if what you call "v6" is this. Segher
