llvm-project.git/llvm/test/Transforms/LowerSwitch, branch users/nico/python-2

[llvm] Remove `br i1 undef` from some regression tests [NFC] (#117112)

2024-11-21T08:06:56+00:00

This PR removes tests with `br i1 undef` under
`llvm/tests/Transforms/Loop*, Lower*`.

[LowerSwitch] Use unsigned integer for range comparison (#93237)

2024-05-27T17:58:22+00:00

Commit 1db51d8eb2 switched from int64_t to `APInt` to prevent high precision integer overflow.
However, when comparing the "range" of switch cases, we should switch to unsigned integer to prevent overflow.
This patch fixes https://github.com/llvm/llvm-project/issues/93152. 
Some test cases are added.

Signed-off-by: Peter Rong

[Pass] Add hyphen to some pass names (#74287)

2024-01-25T08:05:54+00:00

Here is the list of the renamed passes:
- `callbrprepare` -> `callbr-prepare`
- `dwarfehprepare` -> `dwarf-eh-prepare`
- `flattencfg` -> `flatten-cfg`
- `loweratomic` -> `lower-atomic`
- `lowerinvoke` -> `lower-invoke`
- `lowerswitch` -> `lower-switch`
- `winehprepare` -> `win-eh-prepare`
- `targetir` -> `target-ir`
- `targetlibinfo` -> `target-lib-info`

Legacy passes are not affected.

[Transform] Rewrite LowerSwitch using APInt

2023-01-25T04:22:06+00:00

This rewrite fixes https://github.com/llvm/llvm-project/issues/59316.

Previously LowerSwitch uses int64_t, which will crash on case branches using integers with more than 64 bits.
Using APInt fixes this problem. This patch also includes a test

Reviewed By: RKSimon

Differential Revision: https://reviews.llvm.org/D140747

[Transform] Rewrite LowerSwitch using APInt

2023-01-05T22:30:42+00:00

This rewrite fixes https://github.com/llvm/llvm-project/issues/59316.

Previously LowerSwitch uses int64_t, which will crash on case branches using integers with more than 64 bits.
Using APInt fixes this problem. This patch also includes a test

Reviewed By: RKSimon

Differential Revision: https://reviews.llvm.org/D140747

[Transforms] Convert some tests to opaque pointers (NFC)

2023-01-05T11:43:45+00:00

These are all tests where conversion worked automatically, and
required no manual fixup.

[test] Switch to use -passes syntax in various test cases

2022-12-01T20:25:59+00:00

LowerSwitch: Avoid inserting NewDefault block

2022-04-14T05:30:56+00:00

The NewDefault was used to simplify the updating of PHI nodes, but it
causes some inefficiency for target that will run structurizer later. For
example, for a simple two-case switch, the extra NewDefault is causing
unstructured CFG like:

        O
       / \
      O   O
     / \ / \
    C1  ND C2
     \  |  /
      \ | /
        D

The change is to avoid the ND(NewDefault) block, that is we will get a
structured CFG for above example like:

        O
       / \
      /   \
     O     O
    / \   / \
   C1  \ /  C2
    \-> D <-/

The IR change introduced by this patch should be trivial to other targets,
so I am doing this unconditionally.

Fall-through among the cases will also cause unstructured CFG, but it need
more work and will be addressed in a separate change.

Reviewed by: arsenm

Differential Revision: https://reviews.llvm.org/D123607

[LowerSwitch][NewPM] Port lowerswitch to NPM

2020-09-16T01:18:31+00:00

Reviewed By: ychen

Differential Revision: https://reviews.llvm.org/D87726

Fix a crash when the default of a switch is removed

2019-06-03T17:54:15+00:00

This patch fixes a problem that occurs in LowerSwitch when a switch statement has a PHI node as its condition, and the PHI node only has two incoming blocks, and one of those incoming blocks is through an unreachable default in the switch statement. When this condition occurs, LowerSwitch holds a pointer to the condition value, but removes the switch block as a predecessor of the PHI block, causing the PHI node to be replaced. LowerSwitch then tries to use its stale pointer to the original condition value, causing a crash.

Differential Revision: https://reviews.llvm.org/D62560

llvm-svn: 362427