[llvm-commits] [PATCH] Documentation: Fix Datalayout defaults documentation.

Stephen McGruer stephen.mcgruer at gmail.com
Fri Jan 4 11:23:42 PST 2013


(Ping, and changed subject to llvm-commits style.)

This patch updates the LangRef documentation to correctly describe the
default datalayout string. The current documentation (
http://llvm.org/docs/LangRef.html#data-layout) and DataLayout::init() (
http://llvm.org/docs/doxygen/html/DataLayout_8cpp_source.html#l00154)
disagree on a number of points.

Can someone check that my interpretation of DataLayout::init() is correct
and (assuming so) commit the patch?

A full description can be found at
http://llvm.org/bugs/show_bug.cgi?id=14673.

Stephen

---------- Forwarded message ----------
From: Stephen McGruer <stephen.mcgruer at gmail.com>
Date: 21 December 2012 14:26
Subject: Patch for bug 14673 (Mismatchs between Datalayout documentation
and code)
To: llvm-commits at cs.uiuc.edu


Dear all,

This patch updates the LangRef documentation to correctly describe the
default datalayout string in LLVM. The current listing has a number of
inaccuracies, the most important of which is probably the mistaken
statement that aggregate alignment defaults to a0:0:1, instead of the true
a0:0:64.

Full description at http://llvm.org/bugs/show_bug.cgi?id=14673.

Stephen
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130104/f7729452/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: datalayout_docs.patch
Type: application/octet-stream
Size: 1281 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130104/f7729452/attachment.obj>


More information about the llvm-commits mailing list