[PATCH] D34377: Support MS builtins using 'long' on LP64

Bruno Cardoso Lopes via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 20 18:21:55 PDT 2017


bruno marked 3 inline comments as done.
bruno added inline comments.


================
Comment at: test/CodeGen/ms-intrinsics-darwin.c:1
+// RUN: %clang_cc1 -ffreestanding -fms-extensions \
+// RUN:         -triple x86_64--darwin -Oz -emit-llvm %s -o - \
----------------
rnk wrote:
> I'd rather generalize `test/CodeGen/ms-intrinsics.c` to cover Darwin as well as Windows & Linux.
ms-intrinsics.c currently includes intrin.h and does not cover linux. Instead I'm going to rename this ms-intrinsics-other.c, test for linux as well and cover the content from pr27892.c.


https://reviews.llvm.org/D34377





More information about the cfe-commits mailing list