[PATCH] D102259: [flang] Limit the depth of expressions that the compiler will handle

Pete Steinfeld via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 12 06:00:28 PDT 2021


PeteSteinfeld added a comment.

In D102259#2752174 <https://reviews.llvm.org/D102259#2752174>, @klausler wrote:

> Instead of imposing an arbitrary limit on the size of an expression it would be better to change the algorithm to handle larger trees with more use of iteration and tail recursion, I think.

I considered getting rid of recursion for expression processing, but it seemed like a relatively big effort.  I'll have to think about the implications of your suggestion.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102259/new/

https://reviews.llvm.org/D102259



More information about the llvm-commits mailing list