[PATCH] D43561: TableGen: Avoid using resolveListElementReference in TGParser
    Artem Belevich via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Thu Feb 22 10:03:38 PST 2018
    
    
  
tra added a comment.
Please add more details. Description says what the patch does, but not why. There must be the reason for this change, but I can only guess what it is.
The major difference appears to be that now we do call resolveReferences() for each element, while resolveElementReferences() in the old code effectively just returns getElement(i). If resolving references is the purpose of the change, then patch description should reflect that. Also, if that's the case, perhaps it would be better to fix the problem in the List->resolveListElementReference() and make it actually do the reference resolution the name implies.
Repository:
  rL LLVM
https://reviews.llvm.org/D43561
    
    
More information about the llvm-commits
mailing list