[PATCH] D122663: Mark identifier prefixes as substitutable

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 18 06:42:20 PDT 2022


erichkeane added a comment.

@rjmccall  is the expert on the Itanium ABI.

Code wise, I have no real comments, though he might.  As far as:

>> Question: does this need to be covered by -fclang-abi-compat= when the prior mangling resulted in names that even llvm-cxxfilt agreed made no sense? (If it is needed, it should be an easy change.)

I believe the answer here is 'yes'.  We also likely need release notes.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122663



More information about the cfe-commits mailing list