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

2020-05-11 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. In D62368#2029870 , @cryptoad wrote: > In D62368#2029752 , @craig.topper > wrote: > > > I noticed when I pulled this morning that this seems to have been pushed as > > a branch to the

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

2020-05-11 Thread Kostya Kortchinsky via Phabricator via cfe-commits
cryptoad added a comment. In D62368#2030005 , @craig.topper wrote: > I've deleted the branch in the github web interface. I didn't know I could do > that. Ok thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://review

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

2020-05-11 Thread Kostya Kortchinsky via Phabricator via cfe-commits
cryptoad added a comment. In D62368#2029752 , @craig.topper wrote: > I noticed when I pulled this morning that this seems to have been pushed as a > branch to the repo > > From https://github.com/llvm/llvm-project > > eb7d32e..63a4fdd master ->

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

2020-05-11 Thread Craig Topper via Phabricator via cfe-commits
craig.topper added a comment. I noticed when I pulled this morning that this seems to have been pushed as a branch to the repo From https://github.com/llvm/llvm-project eb7d32e..63a4fdd master -> origin/master - [new branch] arcpatch-D62368 -> origin/arcpatch-D62368 Repository:

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

2020-05-11 Thread Kostya Kortchinsky via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG9959eb918acf: Add vendor identity check for Hygon Dhyana processor in Scudo (authored by cryptoad). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D62368/new/

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

2020-05-11 Thread Kostya Kortchinsky via Phabricator via cfe-commits
cryptoad added a comment. In D62368#2027895 , @fanjinke wrote: > Hi Cryptoad, > > Could you help me to commit the patch? Because I don't have access. > > Thanks. Done, thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:

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

2020-05-08 Thread Jinke Fan via Phabricator via cfe-commits
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

2020-05-08 Thread Jinke Fan via Phabricator via cfe-commits
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: compiler-r

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

2020-05-07 Thread Kostya Kortchinsky via Phabricator via cfe-commits
cryptoad added a comment. A couple of style issues to address. Comment at: compiler-rt/lib/scudo/scudo_utils.cpp:66 + +#ifndef signature_HYGON_ebx // They are not defined in the gcc. +/* HYGON: "HygonGenuine" */ s/the gcc/gcc/ Comment at: c

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

2020-05-06 Thread Jinke Fan via Phabricator via cfe-commits
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 change