rogfer01 created this revision.
rogfer01 added reviewers: asb, lenary.
Herald added subscribers: cfe-commits, s.egerton, Jim, benna, psnobl, jocewei,
PkmX, rkruppe, the_o, brucehoult, MartinMosbeck, edward-jones, zzheng, MaskRay,
jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, apazos, simoncook,
asb added a comment.
Thank Roger. While we're doing this, I think it would make sense to default to
ilp32d on 32-bit Linux? I know the glibc support etc is less mature than for
RV64, but it seems the sensible thing to do.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https
lenary added inline comments.
Comment at: clang/lib/Driver/ToolChains/Arch/RISCV.cpp:385
+ ? "ilp32"
+ : Triple.getOS() == llvm::Triple::Linux ? "lp64d" : "lp64";
}
Please may you turn this into a set of if-statements? Nesting ternary ope
