[PATCH] D27051: [X86] Add NumRegisterParameters Module Flag

Nirav Davé via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 29 12:30:49 PDT 2017


Doing so would require non-trivial plumbing of NumRegParameters through the
llvm IR.

That said, I don't believe there's any reason that we cannot move all the
flag declarations up to the end of CodeGenModule::CodeGenModule so they're
all in one place again.

-Nirav

On Wed, Mar 29, 2017 at 3:13 PM, Yaron Keren via Phabricator <
reviews at reviews.llvm.org> wrote:

> yaron.keren added a comment.
>
> Most other module flags are added at CodeGenModule::Release().  For
> consistency, could this code be in CodeGenModule::Release() as well?
>
>
> Repository:
>   rL LLVM
>
> https://reviews.llvm.org/D27051
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170329/ed5c3143/attachment.html>


More information about the llvm-commits mailing list