[PATCH] D49408: [SCCP] Don't use markForcedConstant on branch conditions.

[Florian Hahn via Phabricator via llvm-commits]

fhahn accepted this revision.
fhahn added a comment.

LGTM, thanks!



================
Comment at: test/Transforms/SCCP/ipsccp-paramstate-forcedconst.ll:1
+; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
+; RUN: opt < %s -S -ipsccp | FileCheck %s
----------------
nit: forcedconstant is not involved any more, maybe we should drop it from the name to avoid confusion or combine it with return-zapped.ll to something like ipsccp-branch-condition-unknown-initially.ll?


================
Comment at: test/Transforms/SCCP/return-zapped.ll:46
+
+; Call sites in unreachable blocks should not be a problem.
+; CHECK-LABEL: define i1 @test2() {
----------------
Is this test valuable with the new approach?


Repository:
  rL LLVM

https://reviews.llvm.org/D49408