[PATCH] D36364: [AArch64] Add support for a MinGW AArch64 target

[Martell Malone via Phabricator via cfe-commits]

martell added inline comments.


================
Comment at: lib/Driver/ToolChains/MinGW.cpp:129
+  else if (TC.getArch() == llvm::Triple::aarch64)
+    CmdArgs.push_back("arm64pe");
+  else
----------------
I believe the reason I used thumb2pe for the arm triple is because MS gave us a watered down thumb environment.
If I remember correctly there was also an armce environment in binutils called thumbpe because it was thumb 1 mode.
Anyway, I assume aarch64 on windows is a fully fledged arm64 environment that doesn't use thumb mode over arm64?
I know we don't have any real sdk for this but are you able to tell from link.exe martin?
On arm the branch for a dll call would always set the bit for thumb mode in the instruction.
Although when I think about it arm64pe is probably the most suitable name anyway because we already use thumb2pe



https://reviews.llvm.org/D36364