[PATCH] D43150: [tablegen] Recursively evaluate values of variable initializers.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 15 17:10:31 PST 2018


craig.topper added a comment.

Wasn't the point of how this was written before so that we go through a loop in Record::resolveReferencesTo and resolve each field and call setValue. And the VarInit just loops up the value saved by that resolving. Now it looks we by pass those cached results and just recurse all the time. Won't this be a big perf hit?


https://reviews.llvm.org/D43150





More information about the llvm-commits mailing list