[libcxxabi] r292418 - Revert r286788
Jonathan Roelofs via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 18 10:33:58 PST 2017
On 1/18/17 11:26 AM, Richard Smith via cfe-commits wrote:
> On 18 January 2017 at 10:12, Jonathan Roelofs via cfe-commits
> <cfe-commits at lists.llvm.org <mailto:cfe-commits at lists.llvm.org>> wrote:
> Author: jroelofs
> Date: Wed Jan 18 12:12:39 2017
> New Revision: 292418
> URL: http://llvm.org/viewvc/llvm-project?rev=292418&view=rev
> Revert r286788
> The Itanium ABI  specifies that __cxa_demangle accept either:
> 1) symbol names, which start with "_Z"
> 2) type manglings, which do not start with "_Z"
> r286788 erroneously assumes that it should only handle symbols, so
> this patch
> reverts it and adds a counterexample to the testcase.
> 1: https://mentorembedded.github.io/cxx-abi/abi.html#demangler
> Thanks! Just FYI, the ABI document now lives
> here: https://itanium-cxx-abi.github.io/cxx-abi/
Ah, didn't realize the canonical location changed. Should I get someone
to change where this redirect points?
"http://www.codesourcery.com/cxx-abi/: External link to this page"
As-is, that points at the 'mentorembedded.github.io' one.
> Reviewers: zygoloid, EricWF
jonathan at codesourcery.com
CodeSourcery / Mentor Embedded
More information about the cfe-commits