[all-commits] [llvm/llvm-project] 26e41f: [KnownFPClass] Move `fadd`/`fsub` tests to `nofpcl...

[Eric Ross via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 26e41fdf58e9b41a13dc0c22a107100469b59d35
      https://github.com/llvm/llvm-project/commit/26e41fdf58e9b41a13dc0c22a107100469b59d35
  Author: Eric Ross <zerico2005 at gmail.com>
  Date:   2026-09-11 (Fri, 11 Sep 2026)

  Changed paths:
    A llvm/test/Transforms/Attributor/nofpclass-fadd-fsub.ll
    M llvm/test/Transforms/Attributor/nofpclass.ll

  Log Message:
  -----------
  [KnownFPClass] Move `fadd`/`fsub` tests to `nofpclass-fadd-fsub.ll` (#221872)

- Moved the lit tests for `fadd`/`fsub` to their own dedicated file.
- Updated the lit tests for `fadd`/`fsub` from version 2 to version 6.
- `fadd_p0_ftz_daz #3` was actually testing `positivezero|positivezero`,
so I fixed the name of the test to be `fadd_p0_ftpz_dapz #3`. Then added
a separate `fadd_p0_ftz_daz #0` test.
- `nofpclass-fadd-fsub.ll` uses attributes 1-9 except for 6, so I added
`fadd_p0_dynamic_dynamic #6` so I wouldn't have to change the attributes
for other tests (i.e. changing `test #7 --> test #6`, `test #8 --> test
#7`, etc)

The only tests for `fadd` that remain in `nofpclass.ll` are not related
to testing `KnownFPClass::fadd`.

I have split the PR into 3 commits to make it easier to review (4
commits due to the `undef` false positive).

After this PR lands I plan to open this PR
https://github.com/llvm/llvm-project/pull/221876



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications