[PATCH] D70064: [DAGCombiner][TargetLowering] FCOPYSIGN mixed types legality

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 11 14:38:08 PST 2019


efriedma added a comment.

This isn't quite what I was trying to get at.  Whether we emit a table, or use some other underlying mechanism to provide defaults, isn't really important.  The key points here are:

1. There should be a clear query for whether the operation is legal.
2. We should allow generating unsupported operations before LegalizeDAG runs.
3. LegalizeDAG should support transforming illegal operations to legal ones.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70064/new/

https://reviews.llvm.org/D70064





More information about the llvm-commits mailing list