[PATCH] D52240: Partial Fix for PR#38964
Zhihao Yuan via Phabricator
reviews at reviews.llvm.org
Fri Sep 21 14:00:46 PDT 2018
lichray added a comment.
In https://reviews.llvm.org/D52240#1242536, @dim wrote:
> > Symbol added: __ZNSt3__16__itoa8__u32toaEjPc
> > {'type': 'FUNC', 'is_defined': True, 'name': '__ZNSt3__16__itoa8__u32toaEjPc'}
> >
> > Symbol added: __ZNSt3__16__itoa8__u64toaEyPc
> > {'type': 'FUNC', 'is_defined': True, 'name': '__ZNSt3__16__itoa8__u64toaEyPc'}
> >
> >
> > So I assume the ABI is already broken in some way...
>
> This appears to be a consequence of https://reviews.llvm.org/rL338479 by @lichray , but since it only *adds* two functions, it might be harmless...
Yes, I don't have a Mac to test so I'm reluctant to add those symbols when committing. Anyone who have a Mac can just add it.
https://reviews.llvm.org/D52240
More information about the libcxx-commits
mailing list