atanasyan accepted this revision. atanasyan added inline comments. This revision is now accepted and ready to land.
================ Comment at: lib/Driver/ToolChains/FreeBSD.cpp:197 + CmdArgs.push_back("-m"); + CmdArgs.push_back("elf64ltsmip_fbsd"); + break; ---------------- bsdjhb wrote: > atanasyan wrote: > > Does it make a sense to handle N32 ABI case here? > For whatever reason, FreeBSD only supports N32 on big-endian. However, > binutils does ship elf32ltsmipsn32_fbsd linker scripts after all, so I could > handle little-endian N32 here. LGTM with handling the elf32ltsmipsn32_fbsd case. Repository: rC Clang https://reviews.llvm.org/D48507 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits