[PATCH] D133172: [lit] Extend DEFINE/REDEFINE with function substitutions

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

Sorry for the delayed response, and even now I'm not going to respond directly to your latest comments, as I have to get on with some high priority work that I need to finish ahead of a week off next week. I'll try to look back at this when I'm back.

It might be worth raising some of these points on the discourse thread and see if there's any further thoughts on them, as I don't feel confident the two of us represent a real consensus given the interest shown elsewhere!


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

https://reviews.llvm.org/D133172