[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