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

Reply via email to