<div dir="ltr">While you're moving this code around, it should probably follow the convention for namespaces:<div><a href="http://llvm.org/docs/CodingStandards.html#namespace-indentation">http://llvm.org/docs/CodingStandards.html#namespace-indentation</a><br>
</div><div><br></div><div>Otherwise, go for it.</div><div><br></div><div>---</div><div><br></div><div>I seem to recall that Cray wanted something like this at one point, but I think they were solving a different problem.  They were trying to produce LLVM IR from a different C++ frontend and wanted to reuse the unfortunately overly complicated logic in clang for how we map that down to LLVM IR.  I guess this won't help them, since it takes in clang AST nodes, which they don't have.</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Oct 30, 2013 at 9:54 AM, Mark Lacey <span dir="ltr"><<a href="mailto:mark.lacey@apple.com" target="_blank">mark.lacey@apple.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping?<br>
<br>
Given John’s sign-off are there any objections to just committing this at this point?<br>
<br>
Mark<br>
<div><div class="h5"><br>
On Oct 25, 2013, at 3:24 PM, Mark Lacey <<a href="mailto:mark.lacey@apple.com">mark.lacey@apple.com</a>> wrote:<br>
<br>
> Hi cfe-commits -<br>
><br>
> Please review and let me know if you have any feedback. John McCall already did an off-list review but I wanted to give the community an opportunity before committing.<br>
><br>
> Thanks,<br>
><br>
> Mark<br>
><br>
> --------<br>
><br>
> CodeGenABITypes is a wrapper built on top of CodeGenModule that exposes<br>
> some of the functionality of CodeGenTypes (held by CodeGenModule),<br>
> specifically methods that determine the LLVM types appropriate for<br>
> function argument and return values.<br>
><br>
> I addition to CodeGenABITypes.h, CGFunctionInfo.h is introduced, and the<br>
> definitions of ABIArgInfo, RequiredArgs, and CGFunctionInfo are moved<br>
> into this new header from the private headers ABIInfo.h and CGCall.h.<br>
><br>
> Exposing this functionality is one part of making it possible for LLDB<br>
> to determine the actual ABI locations of function arguments and return<br>
> values, making it possible for it to determine this for any supported<br>
> target without hard-coding ABI knowledge in the LLDB code.<br>
> ---<br>
> include/clang/CodeGen/CGFunctionInfo.h  | 361 ++++++++++++++++++++++++++++++++<br>
> include/clang/CodeGen/CodeGenABITypes.h |  79 +++++++<br>
> lib/CodeGen/ABIInfo.h                   | 145 -------------<br>
> lib/CodeGen/CGAtomic.cpp                |   1 +<br>
> lib/CodeGen/CGBuiltin.cpp               |   1 +<br>
> lib/CodeGen/CGCall.cpp                  |   1 +<br>
> lib/CodeGen/CGCall.h                    | 180 ----------------<br>
> lib/CodeGen/CGClass.cpp                 |   1 +<br>
> lib/CodeGen/CGDecl.cpp                  |   1 +<br>
> lib/CodeGen/CGExprCXX.cpp               |   1 +<br>
> lib/CodeGen/CGObjC.cpp                  |   1 +<br>
> lib/CodeGen/CGObjCMac.cpp               |   1 +<br>
> lib/CodeGen/CGObjCRuntime.cpp           |   1 +<br>
> lib/CodeGen/CGVTables.cpp               |   1 +<br>
> lib/CodeGen/CMakeLists.txt              |   1 +<br>
> lib/CodeGen/CodeGenABITypes.cpp         |  69 ++++++<br>
> lib/CodeGen/CodeGenFunction.cpp         |   1 +<br>
> lib/CodeGen/CodeGenTypes.cpp            |   1 +<br>
> lib/CodeGen/TargetInfo.cpp              |   1 +<br>
> 19 files changed, 523 insertions(+), 325 deletions(-)<br>
> create mode 100644 include/clang/CodeGen/CGFunctionInfo.h<br>
> create mode 100644 include/clang/CodeGen/CodeGenABITypes.h<br>
> create mode 100644 lib/CodeGen/CodeGenABITypes.cpp<br>
><br>
</div></div>> <0001-Add-CodeGenABITypes.h-for-use-in-LLDB.patch><br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div>