[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