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

Luís Marques via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 11 14:56:17 PST 2019


luismarques added a comment.

In D70064#1741400 <https://reviews.llvm.org/D70064#1741400>, @efriedma wrote:

> 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.


Thanks for the clarification, that makes sense. I will rework the patch.


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