nickdesaulniers added subscribers: hjl.tools, rjmccall.
nickdesaulniers added a comment.

Thanks for the patch and the work on cross language CFI support!

I wonder if we have precedent for other non-standard extensions to 
`ItaniumMangleContextImpl`?  I wonder if we should perhaps have a distinct 
subclass to denote that this is not the standard mangling scheme.  It would be 
nice perhaps to get this standardized in 
https://itanium-cxx-abi.github.io/cxx-abi/abi.html#mangling.
https://github.com/itanium-cxx-abi/cxx-abi

cc @rjmccall @hjl.tools


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139395

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

Reply via email to