[LLVMdev] DAGCombiner Patch: Allow targets to do combine first.

Sanjiv.Gupta at microchip.com Sanjiv.Gupta at microchip.com
Tue Mar 24 03:17:10 PDT 2009


The DAGCombiner::combine() in its existing form does not allow targets to do anything if the visitNODE() has done something with it.
IMO, the targets should always be allowed to do stuff irrespective of what we want to do in visitNODE().

Attached is a patch for same. Let me know if it sounds ok. I will go ahead and commit it.

- Sanjiv



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20090324/a97418f5/attachment.html>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: DAGcombine_patch.txt
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20090324/a97418f5/attachment.txt>


More information about the llvm-dev mailing list