probinson added a comment.

Hi @ychen I saw you updated the patch description, thanks.
IIUC, the change causes some difference in the interpretation of particular 
template stuff.  Would this be clearer if the BEFORE-15/AFTER-15 test cases 
used the same template construct?  Currently they are different so it's hard to 
work out the effect of the change.  In particular, naively I see different 
mangled names which makes me concerned that there _is_ in fact a net change to 
mangling.

I'll observe that D128745 <https://reviews.llvm.org/D128745> updated the 
release notes, but not in the most useful way; it cites DRs without any 
explanation of what they do, and with no links.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134507

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

Reply via email to