[PATCH] D85242: [AArch64] [Windows] Error out on unsupported symbol locations

Martin Storsjö via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 5 23:28:59 PDT 2020


mstorsjo added a subscriber: hans.
mstorsjo added a comment.

@hans - WDYT about backporting this one? It's a little bit risky if the check would turn out to be too strict, but there's been no false positives in my test build corpus (which includes a fair bit of assembly source); the upside is for anybody working on porting (mainly assembly) code that isn't originally written for arm64/windows - as it previously wouldn't error out at build time on constructs that don't exist on windows and won't work as intended at runtime.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85242



More information about the llvm-commits mailing list