search for: assignord

...ior after r177525? I recently fixed several places where ordering was not propagated, including during legalization. There are probably still cases that are missed, but I'd be interested in seeing a missed case. I'm guessing it's a legalization that expands to multiple new nodes. The AssignOrdering calls in the legalizer may need to be expanded to do the traversal you mention from SelectionDAGBuilder. On Tue, Mar 26, 2013 at 11:00 PM, Guo, Xiaoyi <Xiaoyi.Guo at amd.com> wrote: > Hi,**** > > ** ** > > It seems orderings are not assigned to DAG nodes created after...

...ated after the DAG builder, e.g., DAG nodes created during legalization. This causes instructions being scheduled in different order than source order for O0. This is my plan to fix it: Make a utility routine that recursively assign ordering to a chain of nodes, just like what SelectionDAGBuilder::AssignOrderingToNode() does. Then add call to this utility routine at each routine that creates new DAG nodes after DAG builder to transfer the original node's order to newly created node. If that routine creates a chain of nodes, then I only need to call the utility routine for the last of the chain. T...

We have some custom nodes that are lowered and we are running into a problem where the Node Ordering is not copied correctly between the original node and the custom lowering. My question is, do we have to manually copy over the ordering in custom nodes via DAG.AssignOrdering, or is there a location that should do this copy for us? Thanks, Micah -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20110606/9bf23e6e/attachment.html>