[PATCH] D41134: [compiler-rt] [builtins] Implement __chkstk for arm64 windows
Michał Górny via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Dec 13 10:53:27 PST 2017
mgorny added inline comments.
================
Comment at: lib/builtins/aarch64/chkstk.S:14
+
+#ifdef __aarch64__
+
----------------
compnerd wrote:
> Im not sure what the point of this check is. This is an AArch64 source so would only be compiled for that target.
We do this in all (most?) of the files for consistency. There were historically cases when tools compiled all files found in the directory — it mostly applies to tests in pre-lit times.
https://reviews.llvm.org/D41134
More information about the llvm-commits
mailing list