[PATCH] D62368: Add vendor identity check for Hygon Dhyana processor in Scudo

fanjinke added a comment. Hi Cryptoad， Could you help me to commit the patch? Because I don't have access. Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62368/new/ https://reviews.llvm.org/D62368

[PATCH] D62368: Add vendor identity check for Hygon Dhyana processor in Scudo

fanjinke updated this revision to Diff 262839. fanjinke added a comment. Changelog: v4: Change the comments to C++ style. Thanks Cryptoad. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62368/new/ https://reviews.llvm.org/D62368 Files:

[PATCH] D62368: Add vendor identity check for Hygon Dhyana processor in Scudo

fanjinke updated this revision to Diff 262529. fanjinke retitled this revision from "Add support for Hygon Dhyana processor" to "Add vendor identity check for Hygon Dhyana processor in Scudo". fanjinke edited the summary of this revision. fanjinke added a comment. Changelog: v3: Remove the

[PATCH] D78874: [clang] Add vendor identity for Hygon Dhyana processor

fanjinke added a comment. Hi Craig， I can't commit the patch myself because I don't have access. Could you please help me？ Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78874/new/ https://reviews.llvm.org/D78874

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke marked an inline comment as done. fanjinke added inline comments. Comment at: compiler-rt/lib/scudo/scudo_utils.cpp:85 + (Ecx == signature_HYGON_ecx); + if (!IsIntel && !IsAMD && !IsHygon) return false; craig.topper wrote: >

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke marked an inline comment as done. fanjinke added inline comments. Comment at: compiler-rt/lib/scudo/scudo_utils.cpp:85 + (Ecx == signature_HYGON_ecx); + if (!IsIntel && !IsAMD && !IsHygon) return false; craig.topper wrote: >

[PATCH] D78874: [clang] Add vendor identity for Hygon Dhyana processor

fanjinke added reviewers: joerg, rsmith. fanjinke added a comment. Hi, Please help with the patch review. Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78874/new/ https://reviews.llvm.org/D78874

[PATCH] D78874: [clang] Add vendor identity for Hygon Dhyana processor

fanjinke added a comment. Hi Clayborg, Thanks for your replay! I will add more reviewers who have previously reviewed this file. Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78874/new/ https://reviews.llvm.org/D78874

[PATCH] D78874: [clang] Add vendor identity for Hygon Dhyana processor

fanjinke created this revision. Herald added subscribers: cfe-commits, cryptoad. Herald added a project: clang. fanjinke added a reviewer: clayborg. fanjinke added a comment. Hi Clayborg, Please help with the patch review. Thanks! The vendor id is used to determine whether the processor

[PATCH] D78874: [clang] Add vendor identity for Hygon Dhyana processor

fanjinke added a reviewer: clayborg. fanjinke added a comment. Hi Clayborg, Please help with the patch review. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78874/new/ https://reviews.llvm.org/D78874

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke added a comment. Hi Cryptoad, Thank you so much for your comment. I will divide it into two patches according to your suggestion. Best regards. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62368/new/ https://reviews.llvm.org/D62368

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke updated this revision to Diff 259150. fanjinke edited the summary of this revision. fanjinke added a comment. 1，Update patch base on lastest commit e90fb82f0f760703c14eafbad96c08b6019a2f0f . 2，Format the patch with

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke added a comment. Hi cryptoad， Any suggestions? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62368/new/ https://reviews.llvm.org/D62368 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke edited reviewers, added: cryptoad; removed: 01alchemist, 4tXJ7f. fanjinke added a comment. Hi Cryptoad, Thanks for your reminds, and I get the points now. Then updated the scudo part of the patch. After patch update，compiler-rt can be successfully compiled using gcc. Thanks again!

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke updated this revision to Diff 250996. fanjinke added a comment. Results of "make check": Testing Time: 1153.40s Expected Passes: 36042 Expected Failures : 163 Unsupported Tests : 340 [100%] Built target check-llvm Scanning dependencies of target check [100%]

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke added reviewers: 01alchemist, 4tXJ7f. fanjinke added a comment. Hi, Please help with the patch reveiw. Hi cryptoad, @cryptoad IS there anything incorrectly? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62368/new/

[PATCH] D62368: Add support for Hygon Dhyana processor

fanjinke created this revision. Herald added projects: clang, Sanitizers, LLVM. Herald added subscribers: llvm-commits, Sanitizers, cfe-commits. This patch adds vendor id detection for Hygon Dhyana CPUs. More details can be found on: