[PATCH] D100596: [WebAssembly] Remove saturating fp-to-int target intrinsics

2021-04-16 Thread Thomas Lively via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5c729750a6d7: [WebAssembly] Remove saturating fp-to-int target intrinsics (authored by tlively). Changed prior to commit: https://reviews.llvm.org

[PATCH] D100596: [WebAssembly] Remove saturating fp-to-int target intrinsics

2021-04-16 Thread Thomas Lively via Phabricator via cfe-commits
tlively added inline comments. Comment at: llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp:124-127 + if (Subtarget->hasNontrappingFPToInt()) +for (auto Op : {ISD::FP_TO_SINT_SAT, ISD::FP_TO_UINT_SAT}) + for (auto T : {MVT::i32, MVT::i64}) +setOperationAc

[PATCH] D100596: [WebAssembly] Remove saturating fp-to-int target intrinsics

2021-04-16 Thread Heejin Ahn via Phabricator via cfe-commits
aheejin accepted this revision. aheejin added inline comments. This revision is now accepted and ready to land. Comment at: llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp:124-127 + if (Subtarget->hasNontrappingFPToInt()) +for (auto Op : {ISD::FP_TO_SINT_SAT, ISD::FP

[PATCH] D100596: [WebAssembly] Remove saturating fp-to-int target intrinsics

2021-04-15 Thread Thomas Lively via Phabricator via cfe-commits
tlively created this revision. tlively added reviewers: aheejin, dschuff. Herald added subscribers: wingo, ecnelises, sunfish, hiraditya, jgravelle-google, sbc100. tlively requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits, cfe-commits. U