[PATCH] D35168: [mips] Handle the `long-calls` feature flags in the MIPS backend
Simon Dardis via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 11 08:12:19 PDT 2017
sdardis added a comment.
2 nits: We should warn/error in LLVM on mixing abicalls and long calls (abicalls overrides long calls). We should make sure clang warns but doesn't error.
Second is about 64bit support, inlined,
================
Comment at: lib/Target/Mips/MipsISelLowering.cpp:3024
+ if (Subtarget.useLongCalls() && Subtarget.hasSym32() && !IsPIC) {
+ // Get the address of the callee into a register to prevent
----------------
Nit:// FIXME: Add support for 64 bit symbols.
Repository:
rL LLVM
https://reviews.llvm.org/D35168
More information about the llvm-commits
mailing list