[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
|
