[LLVMdev] i1 promotion issue (again)

Eli Friedman eli.friedman at gmail.com
Fri Dec 12 13:13:46 PST 2008


On Fri, Dec 12, 2008 at 12:42 PM, Duncan Sands <baldrick at free.fr> wrote:
> Hi Eli,
>
>> Have you tried implementing computeMaskedBitsForTargetNode for your
>> setcc nodes?  If you have, I think DAGCombiner should take care of the
>> necessary simplification.
>
> he doesn't need to: the DAG combiner knows all about SetCC values,
> and should simplify this already.

Oh, this is ISD::SETCC?  SelectionDAG::ComputeNumSignBits doesn't
currently know how to handle it, but that should be easy to fix.

-Eli



More information about the llvm-dev mailing list