[PATCH] D58103: [llvm-readobj] Only allow 4-byte pr_data and refactor GNU_PROPERTY_X86_FEATURE_1_AND processing a bit
George Rimar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Feb 12 00:55:11 PST 2019
grimar added a comment.
Looks correct, FTR ABI link is https://github.com/hjl-tools/x86-psABI/wiki/x86-64-psABI-draft.pdf.
Regarding the refactoring, in my opinion, the original code was simpler. I would leave it as is.
(at least refactoring change could be in a separate patch, if you want to do it)
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58103/new/
https://reviews.llvm.org/D58103
More information about the llvm-commits
mailing list