[libcxxabi] r292418 - Revert r286788

[Jonathan Roelofs via cfe-commits]

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
>     <http://llvm.org/viewvc/llvm-project?rev=292418&view=rev>
>     Log:
>     Revert r286788
>
>     The Itanium ABI [1] 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
>     <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.


Jon

>
>     Reviewers: zygoloid, EricWF
>
>     Modified:
>         libcxxabi/trunk/src/cxa_demangle.cpp
>         libcxxabi/trunk/test/test_demangle.pass.cpp
>


-- 
Jon Roelofs
jonathan at codesourcery.com
CodeSourcery / Mentor Embedded