[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-25 Thread via cfe-commits
https://github.com/martinboehme closed https://github.com/llvm/llvm-project/pull/70046 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-25 Thread Stanislav Gatev via cfe-commits
https://github.com/sgatev approved this pull request. https://github.com/llvm/llvm-project/pull/70046 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread via cfe-commits
@@ -184,6 +192,12 @@ class DataflowAnalysisContext { addTransitiveFlowConditionConstraints(Atom Token, llvm::SetVector &Out); + /// Returns true if the solver is able to prove that there is a satisfying + /// assignment for `Constrai

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread via cfe-commits
@@ -546,12 +546,29 @@ class Environment { Atom getFlowConditionToken() const { return FlowConditionToken; } /// Record a fact that must be true if this point in the program is reached. - void addToFlowCondition(const Formula &); + void assume(const Formula &); + + /// D

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread via cfe-commits
@@ -129,9 +129,17 @@ class DataflowAnalysisContext { /// token. Atom joinFlowConditions(Atom FirstToken, Atom SecondToken); - /// Returns true if and only if the constraints of the flow condition - /// identified by `Token` imply that `Val` is true. - bool flowCondition

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread via cfe-commits
https://github.com/martinboehme updated https://github.com/llvm/llvm-project/pull/70046 >From 8d1119c97581fe5bf9d44330ce2a58b2be18e8b7 Mon Sep 17 00:00:00 2001 From: Martin Braenne Date: Tue, 24 Oct 2023 14:13:44 + Subject: [PATCH 1/2] [clang][dataflow] Add `Environment::allows()`. This al

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread via cfe-commits
martinboehme wrote: CI failures look unrelated https://github.com/llvm/llvm-project/pull/70046 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread Gábor Horváth via cfe-commits
https://github.com/Xazax-hun approved this pull request. https://github.com/llvm/llvm-project/pull/70046 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread Stanislav Gatev via cfe-commits
@@ -129,9 +129,17 @@ class DataflowAnalysisContext { /// token. Atom joinFlowConditions(Atom FirstToken, Atom SecondToken); - /// Returns true if and only if the constraints of the flow condition - /// identified by `Token` imply that `Val` is true. - bool flowCondition

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread Stanislav Gatev via cfe-commits
@@ -546,12 +546,29 @@ class Environment { Atom getFlowConditionToken() const { return FlowConditionToken; } /// Record a fact that must be true if this point in the program is reached. - void addToFlowCondition(const Formula &); + void assume(const Formula &); + + /// D

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread Stanislav Gatev via cfe-commits
@@ -184,6 +192,12 @@ class DataflowAnalysisContext { addTransitiveFlowConditionConstraints(Atom Token, llvm::SetVector &Out); + /// Returns true if the solver is able to prove that there is a satisfying + /// assignment for `Constrai

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread Stanislav Gatev via cfe-commits
sgatev wrote: LGTM https://github.com/llvm/llvm-project/pull/70046 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: None (martinboehme) Changes This allows querying whether, given the flow condition, a certain formula still has a solution (though it is not necessarily implied by the flow condition, as `flowConditionImplies()` would check). This can be c

[clang] [clang][dataflow] Add `Environment::allows()`. (PR #70046)

2023-10-24 Thread via cfe-commits
https://github.com/martinboehme created https://github.com/llvm/llvm-project/pull/70046 This allows querying whether, given the flow condition, a certain formula still has a solution (though it is not necessarily implied by the flow condition, as `flowConditionImplies()` would check). This can