Re: r278483 - This patch implements PR#22821.

2016-10-03 Thread Joerg Sonnenberger via cfe-commits
On Mon, Oct 03, 2016 at 07:10:08AM +, Roger Ferrer Ibanez wrote: > Hi Joerg, > > thanks for your comments. I agree that these false positives are annoying. > > I submitted a while ago a patch to address those false positives in > https://reviews.llvm.org/D23657 but it is pending approval. I'

RE: r278483 - This patch implements PR#22821.

2016-10-03 Thread Roger Ferrer Ibanez via cfe-commits
berger [mailto:jo...@bec.de] > Sent: 02 October 2016 07:24 > To: cfe-commits@lists.llvm.org > Cc: Roger Ferrer Ibanez > Subject: Re: r278483 - This patch implements PR#22821. > > On Sat, Oct 01, 2016 at 11:28:39PM +0200, Joerg Sonnenberger via cfe- > commits wrote: > > On Fri,

Re: r278483 - This patch implements PR#22821.

2016-10-01 Thread Joerg Sonnenberger via cfe-commits
On Sat, Oct 01, 2016 at 11:28:39PM +0200, Joerg Sonnenberger via cfe-commits wrote: > On Fri, Aug 12, 2016 at 08:04:13AM -, Roger Ferrer Ibanez via cfe-commits > wrote: > > Author: rogfer01 > > Date: Fri Aug 12 03:04:13 2016 > > New Revision: 278483 > > > > URL: http://llvm.org/viewvc/llvm-p

Re: r278483 - This patch implements PR#22821.

2016-10-01 Thread Joerg Sonnenberger via cfe-commits
On Fri, Aug 12, 2016 at 08:04:13AM -, Roger Ferrer Ibanez via cfe-commits wrote: > Author: rogfer01 > Date: Fri Aug 12 03:04:13 2016 > New Revision: 278483 > > URL: http://llvm.org/viewvc/llvm-project?rev=278483&view=rev > Log: > This patch implements PR#22821. > > Taking the address of a pa

Re: r278483 - This patch implements PR#22821.

2016-08-23 Thread Aaron Ballman via cfe-commits
test > patch. If there are problems, we can always revert again if needed > > > > Kind regards, > > Roger > > > > From: tha...@google.com [mailto:tha...@google.com] On Behalf Of Nico Weber > Sent: 22 August 2016 17:59 > To: Roger Ferrer Ibanez > Cc: cfe-commi

Re: r278483 - This patch implements PR#22821.

2016-08-23 Thread Nico Weber via cfe-commits
, we can always revert again if needed > > > > Kind regards, > > Roger > > > > *From:* tha...@google.com [mailto:tha...@google.com] *On Behalf Of *Nico > Weber > *Sent:* 22 August 2016 17:59 > *To:* Roger Ferrer Ibanez > *Cc:* cfe-commits > *Subject:* Re: r27

RE: r278483 - This patch implements PR#22821.

2016-08-23 Thread Roger Ferrer Ibanez via cfe-commits
ger Ferrer Ibanez Cc: cfe-commits Subject: Re: r278483 - This patch implements PR#22821. I don't see any approval after Aaron's "Please wait until someone has had the chance to review before committing" on https://reviews.llvm.org/D20561 -- was this reviewed on IRC? On Fr

Re: r278483 - This patch implements PR#22821.

2016-08-22 Thread Nico Weber via cfe-commits
I don't see any approval after Aaron's "Please wait until someone has had the chance to review before committing" on https://reviews.llvm.org/D20561 -- was this reviewed on IRC? On Fri, Aug 12, 2016 at 4:04 AM, Roger Ferrer Ibanez via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: rog

r278483 - This patch implements PR#22821.

2016-08-12 Thread Roger Ferrer Ibanez via cfe-commits
Author: rogfer01 Date: Fri Aug 12 03:04:13 2016 New Revision: 278483 URL: http://llvm.org/viewvc/llvm-project?rev=278483&view=rev Log: This patch implements PR#22821. Taking the address of a packed member is dangerous since the reduced alignment of the pointee is lost. This can lead to memory ali