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

Joel E. Denny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 29 09:09:22 PDT 2022


jdenny added a comment.

In D133172#3822847 <https://reviews.llvm.org/D133172#3822847>, @jhenderson wrote:

> 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.

No worries.  Thanks for the conversation so far.  I should also focus on other things for the moment.

> 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!

That sounds like a good plan.  I'll try to pursue it soon.


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

https://reviews.llvm.org/D133172



More information about the llvm-commits mailing list