This is an archive of the discontinued LLVM Phabricator instance.

[llvm-readelf] Elf e_machine description fix for EM_CR16
ClosedPublic

Authored by sperezglz on Apr 22 2021, 9:34 PM.

Details

Summary

The textual description for EM_CR16 machine type says "Xilinx MicroBlaze", this doesn't seem right and/or intentional. I created this patch to add the right description if desired.

Diff Detail

Event Timeline

sperezglz created this revision.Apr 22 2021, 9:34 PM
sperezglz requested review of this revision.Apr 22 2021, 9:34 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 22 2021, 9:34 PM
MaskRay accepted this revision.Apr 22 2021, 10:05 PM

Thanks!

This revision is now accepted and ready to land.Apr 22 2021, 10:05 PM

@MaskRay I do not have commit access, I will need help, if possible, to get the patch committed. Thanks!

Adding a couple of reviewers more, also I want to point out that I do not have commit access. If reviews go well, kindly asking if someone could commit this on my behalf :) .

Adding a couple of reviewers more, also I want to point out that I do not have commit access. If reviews go well, kindly asking if someone could commit this on my behalf :) .

Can you provide your name and email so that we can attribute the contribution properly?

Author: Sergio Perez Gonzalez <sperezglz@gmail.com>

Kind regards,
Sergio

This revision was landed with ongoing or failed builds.May 2 2021, 7:25 PM
This revision was automatically updated to reflect the committed changes.