probinson added a comment.

In D128934#3630800 <https://reviews.llvm.org/D128934#3630800>, @probinson wrote:

> In D128934#3629653 <https://reviews.llvm.org/D128934#3629653>, @pengfei wrote:
>
>> 
>
> Done, thanks for the reminder!
>
>>> 2. Update `clang/lib/Headers/cpuid.h` and `llvm/lib/Support/Host.cpp` if 
>>> there's a new bit for it;
>
> Not done; I had considered this, but the bit lives in a CPUID leaf that is 
> not currently supported at all by the header or Host.cpp.  I'm willing to add 
> RDPRU, with FIXMEs for the other feature bits, if people want it.

@RKSimon pointed out to me that I was mistaken about this.  Not sure how I 
missed it...  I'm about to go on holiday so he is authorized to clean up my 
mess.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D128934/new/

https://reviews.llvm.org/D128934

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to