Skip to content

Conversation

@gigalasr
Copy link
Collaborator

@gigalasr gigalasr commented Nov 11, 2024

Also fixes #1150

@gigalasr gigalasr self-assigned this Nov 11, 2024
@gigalasr gigalasr linked an issue Nov 11, 2024 that may be closed by this pull request
@gigalasr
Copy link
Collaborator Author

gigalasr commented Nov 11, 2024

  • flowr/src/dataflow/internal/process/functions/call/built-in/built-in-if-then-else.ts anpassen
    • neue Funktion, checken ob alle defs true bzw false sind
  • tests schreiben
    • 2 dataflow,
    • 2 slicing tets
    • simple.test.ts als bsp.
    • slicing tests in control-flow.tests.ts
    • dataflow tests in dataflow-if-then.tests.ts

@gigalasr gigalasr requested a review from EagleoutIce November 18, 2024 08:21
@EagleoutIce EagleoutIce merged commit 3cdffb0 into main Nov 18, 2024
@EagleoutIce EagleoutIce deleted the 1102-abstract-domains-ii branch November 18, 2024 08:37
manueldiagostino pushed a commit to manueldiagostino/flowr that referenced this pull request Nov 20, 2024
* feat(resolve-by-name): resolve to list of constants (flowr-analysis#1102)

* refactor(resolve-test): use test.each

* lint(resolve-test): fix es-lint issues

* feat(buil-in): use new resolveToConstatns function (flowr-analysis#1102)

* test(resolve): add tests for new resolveToConstants function (flowr-analysis#1102)

* test(control-flow): test dead code detection (flowr-analysis#1102)

* test-fix(control-flow): fix naming (flowr-analysis#1102)

* test(df-if-then-else): dead code test (flowr-analysis#1102)

* feat-fix(ite): strengthen env merge in case of constant cases

* test-fix(cft): add missing statement (flowr-analysis#1102)

* test(dite): test entire graph fro dead code (flowr-analysis#1102)

* test-fix(ditt): provide correct graph for testing (flowr-analysis#1102)

---------

Co-authored-by: Florian Sihler <florian.sihler@uni-ulm.de>
@EagleoutIce
Copy link
Member

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Overwritten variables are still shown in slice [Abstract Domains, II] Improve Dead-Code Identification

3 participants