Re: svn commit: r357337 - head/sys/riscv/include

2020-01-31 Thread John Baldwin
On 1/31/20 1:17 PM, Conrad Meyer wrote: > Hi John, > > Isn't the 32-bit MSTATUS_SD and SSTATUS_SD also UB without explicit > unsigned suffix? Possibly, but I doubt we will have a 32-bit RISC-V port. I'd probably prefer to just use explicit hex values (0x8000, etc.) since the compiler will al

Re: svn commit: r357337 - head/sys/riscv/include

2020-01-31 Thread Conrad Meyer
Hi John, Isn't the 32-bit MSTATUS_SD and SSTATUS_SD also UB without explicit unsigned suffix? Best, Conrad On Fri, Jan 31, 2020 at 9:49 AM John Baldwin wrote: > > Author: jhb > Date: Fri Jan 31 17:49:15 2020 > New Revision: 357337 > URL: https://svnweb.freebsd.org/changeset/base/357337 > > Log:

svn commit: r357337 - head/sys/riscv/include

2020-01-31 Thread John Baldwin
Author: jhb Date: Fri Jan 31 17:49:15 2020 New Revision: 357337 URL: https://svnweb.freebsd.org/changeset/base/357337 Log: Fix 64-bit value of SSTATUS_SD to use an unsigned long. While here, fix MSTATUS_SD to match SSTATUS_SD. Reviewed by: mhorne MFC after:2 weeks Sponsored by