According to RISC-V Unprivileged ISA 15.6.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Unit Tests
Unit Tests
Time | Test | |
---|---|---|
2,160 ms | x64 debian > AddressSanitizer-x86_64-linux.TestCases::strcmp.c |
Event Timeline
Comment Actions
From an initial look, I think this is almost ready to land - thank you.
One suggestion would be to review the various Subtarget.hasStdExtZfhmin() || Subtarget.hasStdExtZfh(). I'd first thought adding a new helper to RISCVSubtarget to be called instead, but actually it looks like a number of these could be replaced by just hasStdExtZfhmin due to Zfhmin implying Zfh.
llvm/lib/Target/RISCV/RISCVInstrInfoZfh.td | ||
---|---|---|
187 | Unintended change? |
Comment Actions
LGTM, modulo one tiny nit on a comment. Thanks!
llvm/lib/Target/RISCV/RISCVInstrInfoZfh.td | ||
---|---|---|
253 | HasStdExtZfh=>HasStdExtZfhmin |
This could equivalently just be if (VT == MVT::f16 && !Subtarget.hasStdExtZfhmin()), right?