This is an archive of the discontinued LLVM Phabricator instance.

[clang] Add vendor identity for Hygon Dhyana processor
ClosedPublic

Authored by fanjinke on Apr 25 2020, 11:39 PM.

Details

Summary

The vendor id is used to determine whether the processor
supports hardware CRC32 in the Scudo code. The previous
discussion about the patch is in [1], and more information
about Hygon Dhyana processor is in[2].

Result of "make check":
Testing Time: 1339.56s

Unsupported Tests:   317
Expected Passes  : 36674
Expected Failures:   161

[100%] Built target check-llvm
[100%] Built target check

[1]: https://reviews.llvm.org/D62368
[2]: https://git.kernel.org/torvalds/c/c9661c1e80b609cd038db7c908e061f0535804ef

Signed-off-by: Jinke Fan <fanjinke51@yeah.net>

Diff Detail

Event Timeline

fanjinke created this revision.Apr 25 2020, 11:39 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 25 2020, 11:39 PM

Hi Clayborg,

Please help with the patch review.

Thanks!

I am not sure I am the right person to review? I work on LLDB and LLVM mostly.

Hi Clayborg,

Thanks for your replay!
I will add more reviewers who have previously reviewed this file.

Thanks.

Hi,
Please help with the patch review.

Thanks.

craig.topper accepted this revision.Apr 26 2020, 8:05 PM
craig.topper added a subscriber: craig.topper.

LGTM

This revision is now accepted and ready to land.Apr 26 2020, 8:05 PM

Hi Craig,

I can't commit the patch myself because I don't have access.
Could you please help me?

Thanks.

This revision was automatically updated to reflect the committed changes.