[PATCH] D65978: [clang] Fixed x86 cpuid NSC signature

This revision was automatically updated to reflect the committed changes. Closed by commit rL368510: [clang] Fixed x86 cpuid NSC signature (authored by teemperor, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm

[PATCH] D65978: [clang] Fixed x86 cpuid NSC signature

teemperor accepted this revision. teemperor added a comment. LGTM. Thanks for the patch! Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65978/new/ https://reviews.llvm.org/D65978 ___ cfe-commits mailing list cfe-commi

[PATCH] D65978: [clang] Fixed x86 cpuid NSC signature

Jacotot added a comment. In D65978#1624220 , @teemperor wrote: > @Jacotot I assume you don't have commit access to merge yourself? I don't : this is my very first contribution to the project Repository: rC Clang CHANGES SINCE LAST ACTION https://r

[PATCH] D65978: [clang] Fixed x86 cpuid NSC signature

teemperor added a comment. @Jacotot I assume you don't have commit access to merge yourself? Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65978/new/ https://reviews.llvm.org/D65978 ___ cfe-commits mailing list cfe-c

[PATCH] D65978: [clang] Fixed x86 cpuid NSC signature

craig.topper accepted this revision. craig.topper added a comment. This revision is now accepted and ready to land. LGTM Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65978/new/ https://reviews.llvm.org/D65978 ___ cf

[PATCH] D65978: [clang] Fixed x86 cpuid NSC signature

Jacotot created this revision. Jacotot added reviewers: teemperor, rsmith. Herald added a project: clang. Herald added a subscriber: cfe-commits. The signature "Geode by NSC" for NSC vendor is wrong. In lib/Headers/cpuid.h, signature_NSC_edx and signature_NSC_ecx constants are inverted (cpuid sig