This is an archive of the discontinued LLVM Phabricator instance.

[demangler][LoongArch] Correct the mangled_size for long double
ClosedPublic

Authored by SixWeining on Nov 29 2022, 10:43 PM.

Details

Summary

The size of long double in LoongArch (both LA32 and LA64) is 16 bytes, thus
the mangled_size should be 32.

This is same as RISCV's change in D126480.

Diff Detail

Event Timeline

SixWeining created this revision.Nov 29 2022, 10:43 PM
SixWeining requested review of this revision.Nov 29 2022, 10:43 PM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptNov 29 2022, 10:43 PM
Herald added a reviewer: Restricted Project. · View Herald Transcript
xen0n accepted this revision.Nov 29 2022, 10:47 PM

I have made the same change locally but your commit message is somewhat better than mine in that it provides a bit more context. I'll rebase on top of this change then. Thanks!

Can a libc++ approver take a look? Thanks.

This revision was not accepted when it landed; it landed in state Needs Review.Dec 7 2022, 6:22 PM
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.