[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