[PATCH] D60747: Remove EnableEarlyCSEMemSSA option

Tue Apr 16 16:06:33 PDT 2019

george.burgess.iv added a comment.

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.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60747/new/

https://reviews.llvm.org/D60747