This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

RE: [PATCH] icc: allow code path for newer versions of icc.

From: "Tedeschi, Walfred" <walfred dot tedeschi at intel dot com>
To: Pedro Alves <palves at redhat dot com>, Simon Marchi <simon dot marchi at polymtl dot ca>
Cc: "qiyaoltc at gmail dot com" <qiyaoltc at gmail dot com>, "gdb-patches at sourceware dot org" <gdb-patches at sourceware dot org>
Date: Wed, 20 Sep 2017 15:55:18 +0000
Subject: RE: [PATCH] icc: allow code path for newer versions of icc.
Authentication-results: sourceware.org; auth=none
Dlp-product: dlpe-windows
Dlp-reaction: no-action
Dlp-version: 11.0.0.116
References: <1504687613-14649-1-git-send-email-walfred.tedeschi@intel.com> <59e18301-7b12-2e92-6277-0aef4164243e@redhat.com> <d7ed4db4289cc351be5bd0e1be0b16ec@polymtl.ca> <914ad9ed-940e-81e1-e7de-6c018d29aec7@intel.com> <4bc75ec493ddeb1500c51812a51e8182@polymtl.ca> <a83f63ef-65cb-25f1-0cfc-585c21172b18@intel.com> <cd49d371-7265-0e53-9f33-ab53e59f97ea@redhat.com>

-----Original Message-----
From: gdb-patches-owner@sourceware.org [mailto:gdb-patches-owner@sourceware.org] On Behalf Of Pedro Alves
Sent: Wednesday, September 20, 2017 5:46 PM
To: Tedeschi, Walfred <walfred.tedeschi@intel.com>; Simon Marchi <simon.marchi@polymtl.ca>
Cc: qiyaoltc@gmail.com; gdb-patches@sourceware.org
Subject: Re: [PATCH] icc: allow code path for newer versions of icc.

On 09/20/2017 03:17 PM, Tedeschi, Walfred wrote:

> Hello Simon and Pedro,
> 
> I have prepared a cleaning patch first and another one with the icc 
> version check.
> 
> The changes are in:
> users/wtedesch/icc_version
> 
> Comments are welcome!

Thanks!

Speaking for myself, while a git branch is handy for trying locally (and I tend to push changes to branches too to help others for larger series, in addition to posting on the list), I still find it easier to reply/review to patches sent via email, because that allows conveniently quoting patches.

But I went ahead and took a quick look.  I have to say that don't care for the "dwarf2utils" name much BTW.  My issue with "utils.h|c" is that "utils" is a kitchen sync name; almost anything can be called an "utility".  And "dwarf2utils" has almost the same issue, in my view.  Note that I had suggested producers.h/c without a "dwarf" qualification because the producers string is exposed in generic code, through symtab.h:compunit_symtab etc.
(even though the producers info is currently only sourced from DWARF info).

As for the the rest, it seems like several previous comments haven't been addressed yet, so I'll wait for a repost.

Thanks,
Pedro Alves

Pedro,

Thanks for your quick answer!

I will prepare new patches.

Thanks and regards,
/Fred
Intel Deutschland GmbH
Registered Address: Am Campeon 10-12, 85579 Neubiberg, Germany
Tel: +49 89 99 8853-0, www.intel.de
Managing Directors: Christin Eisenschmid, Christian Lamprechter
Chairperson of the Supervisory Board: Nicole Lau
Registered Office: Munich
Commercial Register: Amtsgericht Muenchen HRB 186928

References:
- [PATCH] icc: allow code path for newer versions of icc.
  - From: Walfred Tedeschi
- Re: [PATCH] icc: allow code path for newer versions of icc.
  - From: Pedro Alves
- Re: [PATCH] icc: allow code path for newer versions of icc.
  - From: Simon Marchi
- Re: [PATCH] icc: allow code path for newer versions of icc.
  - From: Tedeschi, Walfred
- Re: [PATCH] icc: allow code path for newer versions of icc.
  - From: Simon Marchi
- Re: [PATCH] icc: allow code path for newer versions of icc.
  - From: Tedeschi, Walfred
- Re: [PATCH] icc: allow code path for newer versions of icc.
  - From: Pedro Alves

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]