[PATCH] D78393: [SCCP] Drop unnecessary early exit for ExtractValueInst.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Apr 17 12:26:14 PDT 2020


fhahn created this revision.
fhahn added reviewers: efriedma, mssimpso, davide.
Herald added a subscriber: hiraditya.
Herald added a project: LLVM.

visitExtractValueInst uses mergeInValue, so it already can handle
constant ranges. Initially the early exit was using isOverdefined to
keep things as NFC during the initial move to ValueLatticeElement.
As the function already supports constant ranges, it can just use
ValueState[&I].isOverdefined.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D78393

Files:
  llvm/lib/Transforms/Scalar/SCCP.cpp
  llvm/test/Transforms/SCCP/conditions-ranges.ll
  llvm/test/Transforms/SCCP/constant-range-struct.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D78393.258394.patch
Type: text/x-patch
Size: 4240 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20200417/e2f1fa03/attachment.bin>


More information about the llvm-commits mailing list