[WebAssembly] Do not remove existing output file.

[ruiu]

Summary

FileOutputBuffer automatically removes an existing file, so we don't
need to do that. Actually doing that is discouraged because when the
linker fails to create an output for some reason after instantiating
FileOutputBufffer, FileOutputBuffer removes a temporary file and don't
touch an existing file. That's an desired behavior from the user's
point of view.

(Internally, FileOutputBuffer writes its contents to a temporary file
and then rename it over to an existing file on commit()).

wasm/Writer.cpp

void Writer::run() {
 
   if (Error E = Buffer->commit())
     fatal("failed to write the output file: " + toString(std::move(E)));
 }
 
 // Open a result file.
 void Writer::openFile() {
   log("writing: " + Config->OutputFile);
-  ::remove(Config->OutputFile.str().c_str());
 
   Expected<std::unique_ptr<FileOutputBuffer>> BufferOrErr =
       FileOutputBuffer::create(Config->OutputFile, FileSize,
                                FileOutputBuffer::F_executable);
 
   if (!BufferOrErr)
     error("failed to open " + Config->OutputFile + ": " +
           toString(BufferOrErr.takeError()));