https://github.com/efriedma-quic commented:

Oh, this is making a bit more sense now... I didn't realize this was overriding 
method that handles char/short already.  And I guess for BPF targets, you can 
assume "long" is 64 bits.

That said, this isn't handling "unsigned int" correctly.  "unsigned int" is 
supposed to be sign-extended on RISC-V targets.  (How you resolve the conflict 
with PowerPC, where "unsigned int" is supposed to be zero-extended, I'm not 
sure.)

https://github.com/llvm/llvm-project/pull/84874
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to