Page MenuHomePhabricator

Please use GitHub pull requests for new patches. Phabricator shutdown timeline

Differential D151992

[RISCV] Avoid illegal DAG combination for strict-fp nodes.
ClosedPublic
Actions

Authored by fakepaper56 on Jun 2 2023, 6:21 AM.

Details

Reviewers
craig.topper
reames
frasercrmck
rogfer01
luke
Commits
rG84ee08c6a89a: [RISCV] Avoid illegal DAG combination for strict-fp nodes.
Summary

Those combines may change the exception behavior and rounding behavior.

Diff Detail

Event Timeline

fakepaper56 created this revision.Jun 2 2023, 6:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 2 2023, 6:21 AM
Herald added subscribers: jobnoorman, VincentWu, vkmr and 25 others. · View Herald Transcript
fakepaper56 requested review of this revision.Jun 2 2023, 6:21 AM
Herald added a project: Restricted Project. · View Herald TranscriptJun 2 2023, 6:21 AM
Herald added subscribers: llvm-commits, • pcwang-thead, eopXD, MaskRay. · View Herald Transcript
Harbormaster completed remote builds in B236162: Diff 527832.Jun 2 2023, 7:26 AM
craig.topper accepted this revision.Jun 2 2023, 9:50 AM

LGTM

This revision is now accepted and ready to land.Jun 2 2023, 9:50 AM
This revision was landed with ongoing or failed builds.Jun 2 2023, 7:29 PM
Closed by commit rG84ee08c6a89a: [RISCV] Avoid illegal DAG combination for strict-fp nodes. (authored by fakepaper56). · Explain Why
This revision was automatically updated to reflect the committed changes.
fakepaper56 added a commit: rG84ee08c6a89a: [RISCV] Avoid illegal DAG combination for strict-fp nodes..

Revision Contents
Changeset List

PathSize
llvm/
lib/
Target/
RISCV/
8 lines

Diff 528067

View Options

llvm/lib/Target/RISCV/RISCVISelLowering.cpp

Loading...