[PATCH] D126425: [llvm-ml] Add support for the .S extension

Mon May 30 06:48:55 PDT 2022

hans added a comment.

It appears to me that MASM doesn't care much about the file extension at all:

  C:\src\tmp>type a.blah
  .code

  main proc
    ret
  main endp

  END
  C:\src\tmp>ml64 a.blah /link /entry:main
  Microsoft (R) Macro Assembler (x64) Version 14.28.29336.0
  Copyright (C) Microsoft Corporation.  All rights reserved.

   Assembling: a.blah
  Microsoft (R) Incremental Linker Version 14.28.29336.0
  Copyright (C) Microsoft Corporation.  All rights reserved.

  /OUT:a.exe
  a.obj
  /entry:main

The fact that `/Ta` exists suggest maybe it did care at some point, but that may have changed.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126425