dblaikie added a comment.

Partly feedback on D139989 <https://reviews.llvm.org/D139989> too - if we could 
move this logic outside the switch, then it wouldn't need to be done in each 
different template parameter type? (& then D139989 
<https://reviews.llvm.org/D139989> I guess wouldn't be needed - this patch and 
the other would be subsumed by removing the old logic from distinct switch 
cases, and putting the new logic that works across different template parameter 
kinds outside the switch - solving things more generically)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139988

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to