[PATCH] D35014: [X86] PR32755 : Improvement in CodeGen instruction selection for LEAs.

James Molloy via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 18 05:04:27 PDT 2017


jmolloy requested changes to this revision.
jmolloy added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/Target/X86/X86OptimizeLEAs.cpp:1029
+  for (auto Child : DN->getChildren())
+    FactorizeLEAsBasicBlock(Child);
+
----------------
This can cause recursion deep enough to cause stack overflows. Please could you refactor this to not use direct recursion? The domtree may be hundreds of nodes deep in degenerate cases.


https://reviews.llvm.org/D35014





More information about the llvm-commits mailing list