[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [XEN PATCH v7 01/51] build: introduce cpp_flags macro

  • To: Jan Beulich <jbeulich@xxxxxxxx>
  • From: Anthony PERARD <anthony.perard@xxxxxxxxxx>
  • Date: Mon, 6 Sep 2021 12:21:06 +0100
  • Authentication-results: esa1.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
  • Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, George Dunlap <george.dunlap@xxxxxxxxxx>, Ian Jackson <iwj@xxxxxxxxxxxxxx>, Julien Grall <julien@xxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>, Roger Pau Monné <roger.pau@xxxxxxxxxx>, "Tim Deegan" <tim@xxxxxxx>, <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • Delivery-date: Mon, 06 Sep 2021 11:21:21 +0000
  • Ironport-hdrordr: A9a23:Ith7NqFoh3gH0OgxpLqEP8eALOsnbusQ8zAXPhhKOH5om7+j5q WTdZUgpGXJYVkqNk3I9erqBEDEewK+yXcX2/htAV7BZmjbUQKTRelfBMnZowEIcBeSygcp78 hdmt9FZOEYY2IRsS+w2njdLz9p+qjhzIm4werRi2xgQmhRGsddxhY8BwjeCUtsS2B9dOME/F nw3Ls9m9OPQwVuUviG
  • Ironport-sdr: xeB6+cUldNGZTvDItGjVErmqjASTCsTVVfkPWK+6B2Rqqv5Hv4n6V7J9v6Tl/NsoMBR8fdwpuy auNR9FNlKJHExAJ6L3DIz5rfs/gdicgXRM71dDSYfZSu9WF1Nzw8llywlQ+QQ6Cuh2nEzOl/pV yPqQH1MTHF5Gb83QuCrgthhX0dk204VXcoZR8aMWhEw6CHS2x014BFOShN7ncfhqPwAnjr+YVZ Orm2iTV2v7dXFCY2Rr48afLqC40tLiL0bNjHZOjwGJf/rsTOi/EnbStdcnfngtzr9Y7MOaJx7+ 2wyRtrKAUQ4db7TVnTxuUO/F
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Mon, Sep 06, 2021 at 12:30:07PM +0200, Jan Beulich wrote:
> On 06.09.2021 12:06, Anthony PERARD wrote:
> > On Thu, Sep 02, 2021 at 12:08:58PM +0200, Jan Beulich wrote:
> >> On 24.08.2021 12:49, Anthony PERARD wrote:
> >>> +# To be use with $(a_flags) or $(c_flags) to produce CPP flags
> >>> +cpp_flags = $(filter-out -Wa$(comma)% -flto,$(1))
> Well, yes. This way one could also pass more than just the expansion of
> either of these two variables. The thing that made me think of the
> alternative is the comment: Would you mind if I inserted "e.g." in there,
> to make clear this isn't limited to these two variables?

Adding "e.g." is fine. Thanks.

Anthony PERARD



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.