[PATCH] D72068: [IR] Refactor SubclassData
    Ehud Katz via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Mar  4 11:15:41 PST 2020
    
    
  
ekatz added a comment.
In D72068#1889825 <https://reviews.llvm.org/D72068#1889825>, @rnk wrote:
> I'm still basically not in favor of this. If another contributor thinks it's a great idea, I wouldn't go so far as to block this change, but I don't plan to do any further review.
Fair enough, I respect your opinion.
However, I still think it is a very welcoming change (for the many reasons listed in the "summary"), and I hope other reviewers will see the benefits as I see them, and approve this change.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72068/new/
https://reviews.llvm.org/D72068
    
    
More information about the llvm-commits
mailing list