[PATCH] D60747: Remove EnableEarlyCSEMemSSA option
Eric Christopher via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 16 16:07:52 PDT 2019
echristo added a comment.
In D60747#1469517 <https://reviews.llvm.org/D60747#1469517>, @george.burgess.iv wrote:
> I can't find any users of this flag, so I'm in favor of s/EnableEarlyCSEMemSSA/true/ if that simplifies things.
>
> > Is this still useful for debugging problems with MemSSA and GVNHoist?
>
> If we just have to tweak this one place, IMO it's sufficiently straightforward to just manually flip between true/false in code. I've personally never used it to debug anything, though I can only speak for myself.
That was pretty much my thought. As far as the new pass manager - the flag isn't being used anywhere else other than this location :)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D60747/new/
https://reviews.llvm.org/D60747
More information about the llvm-commits
mailing list