pengfei added a comment.

Sorry, I missed this patch. I think this is a wrong fix. The `buf[9]` doesn't 
make sense to X86 registers. You can see GCC fails with it too. 
https://godbolt.org/z/vPcfoG714

I think we just need to do sema check for it to emit error gracefully. BTW, I 
have a patch D107141 <https://reviews.llvm.org/D107141> to fix structure 
issues. Welcomes for suggestions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120596

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

Reply via email to