search for: patatino1

> I'm not necessarily sympathetic to the idea of adding another canonicalization pass only for this purpose. The problem is that as you said, SimplifyCfg does all sorts of stuff, and I suspect is not the fastest pass in the world. Also, in the case that annoys me, there is an LCSSA pass in the middle, and I suspect it would be a better idea to only do the LCSSA etc. transform again if no

...get O(N^3). > The alternative SSAupdater proposed solves this problem using the > novel algorithm from Braun et al. linked in the review > https://reviews.llvm.org/D28934 > It still has the problem that if leaves redundant phis around at the > beginning of the block, e.g. > > %patatino1 = phi [ %tinky, %somebb, %winky, %someotherbb ] > %patatino2 = phi [ %tinky, %somebb, %winky, %someotherbb ] > > The current SSA updater pays a (sometimes high) cost to perform this > deduplication/removal (actually, checks whether there's already a PHI, > and if not inserts one)...