[PATCH] D11777: [Target/X86] Allow LowerVAARG to work on FreeBSD, too.
Charles Davis
cdavis5x at gmail.com
Wed Aug 5 13:56:22 PDT 2015
cdavis5x created this revision.
cdavis5x added a reviewer: emaste.
cdavis5x added a subscriber: llvm-commits.
This change enables the `va_arg` instruction to work on
FreeBSD/amd64, too. It's needed to prevent test failures after D1622
lands. Since FreeBSD/amd64 largely follows the System V ABI for x86-64
(like Linux/x86-64), I don't see any reason why this wouldn't work.
http://reviews.llvm.org/D11777
Files:
lib/Target/X86/X86ISelLowering.cpp
Index: lib/Target/X86/X86ISelLowering.cpp
===================================================================
--- lib/Target/X86/X86ISelLowering.cpp
+++ lib/Target/X86/X86ISelLowering.cpp
@@ -15046,6 +15046,7 @@
assert(Subtarget->is64Bit() &&
"LowerVAARG only handles 64-bit va_arg!");
assert((Subtarget->isTargetLinux() ||
+ Subtarget->isTargetFreeBSD() ||
Subtarget->isTargetDarwin()) &&
"Unhandled target in LowerVAARG");
assert(Op.getNode()->getNumOperands() == 4);
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11777.31392.patch
Type: text/x-patch
Size: 520 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150805/65d30dce/attachment.bin>
More information about the llvm-commits
mailing list