Re: [PATCH] D17453: [Driver] Enable --rtlib option for MSVC target

[Roman Shirokiy via cfe-commits]

roman.shirokiy added a comment.

Hello!

Thanks for the feedback!

MSVC got its own runtime lib, so the function "AddRunTimeLibs" is not used 
anywhere in "visualstudio::Linker::Constructjob" i.e. "--rtlib" option is 
currently unused in MSVC environment (there is always a warning: argument 
unused during compilation').

  if (!Args.hasArg(options::OPT_nostdlib)) {
    AddRunTimeLibs(TC, TC.getDriver(), CmdArgs, Args);
  }

This change is necessary to actually handle "--rtlib" on MSVC, but I totally 
agree that test for "-nostdlib --rtlib=compiler-rt" case is missed.


http://reviews.llvm.org/D17453



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits