[PATCH] D154176: [Driver][MSVC] Move lld specific flags before inputs

Sat Jul 1 02:37:45 PDT 2023

mstorsjo added a comment.

In D154176#4466186 <https://reviews.llvm.org/D154176#4466186>, @mstorsjo wrote:

> Ok, thanks for clarifying. However I still don’t understand the “why” aspect here. You’re writing
>
>> so that lld specific flags can be append before inputs
>
> Are you planning on adding such flags in a later patch, position dependent flags that need to be supplied before input files?
>
> Or does the change in order of command line arguments have an effect on the behavior of the linker in this case?

Ok, on second thought, I guess that parsing the vfsoverlay option before the inputs makes it have an effect where it didn’t before. Is that right? (Most flags in lld aren’t very order dependent since it just checks for any occurrence of a flag anywhere among the arguments.)

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D154176/new/

https://reviews.llvm.org/D154176