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

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 11 14:00:13 PDT 2019


davide added a comment.

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.


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