[PATCH] D60581: [ValueLattice] Add forceconstant, for SCCP.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 11 14:08:39 PDT 2019


fhahn added a comment.

In D60581#1463363 <https://reviews.llvm.org/D60581#1463363>, @davide wrote:

> I'm not necessarily sure I like this. It's simply spreading in a general framework a concept that shouldn't exist to begin with. I'd rather remove forced constant from SCCP, even though it might result in us losing some optimization power.


Right, I am happy to go in that direction as well.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D60581





More information about the llvm-commits mailing list