nathanchance added a comment.

In D110869#3302625 <https://reviews.llvm.org/D110869#3302625>, @nickdesaulniers 
wrote:

> All built+boot. I know @nathanchance pointed out an issue with some already 
> complex code, but I'm of the opinion that fn should be attributed in kernel 
> sources.

Aside from that one diff I posted above adding 
`__attribute__((zero_call_used_regs("skip")))` to `_paravirt_ident_64()`, I 
have not noticed any issues with diff        406596 on my two test machines; 
they boot right up and I am currently running them through my usual build 
tests. I will comment back if anything bad turns up.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110869

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

Reply via email to