Re: r309106 - Recommit r308327 2nd time: Add a warning for missing

2017-07-28 Thread Alex L via cfe-commits
Thanks, I recommitted it in r309386. On 27 July 2017 at 15:53, Hans Wennborg wrote: > On Thu, Jul 27, 2017 at 3:41 AM, Alex L wrote: > > > > > > On 26 July 2017 at 22:32, Hans Wennborg wrote: > >> > >> On Wed, Jul 26, 2017 at 11:27 AM, Hans Wennborg > wrote: > >> > On Wed, Jul 26, 2017 at 5:2

Re: r309106 - Recommit r308327 2nd time: Add a warning for missing

2017-07-27 Thread Hans Wennborg via cfe-commits
On Thu, Jul 27, 2017 at 3:41 AM, Alex L wrote: > > > On 26 July 2017 at 22:32, Hans Wennborg wrote: >> >> On Wed, Jul 26, 2017 at 11:27 AM, Hans Wennborg wrote: >> > On Wed, Jul 26, 2017 at 5:20 AM, Alex Lorenz via cfe-commits >> > wrote: >> >> Author: arphaman >> >> Date: Wed Jul 26 05:20:57 2

Re: r309106 - Recommit r308327 2nd time: Add a warning for missing

2017-07-27 Thread Alex L via cfe-commits
On 26 July 2017 at 22:32, Hans Wennborg wrote: > On Wed, Jul 26, 2017 at 11:27 AM, Hans Wennborg wrote: > > On Wed, Jul 26, 2017 at 5:20 AM, Alex Lorenz via cfe-commits > > wrote: > >> Author: arphaman > >> Date: Wed Jul 26 05:20:57 2017 > >> New Revision: 309106 > >> > >> URL: http://llvm.org/

Re: r309106 - Recommit r308327 2nd time: Add a warning for missing

2017-07-26 Thread Hans Wennborg via cfe-commits
On Wed, Jul 26, 2017 at 11:27 AM, Hans Wennborg wrote: > On Wed, Jul 26, 2017 at 5:20 AM, Alex Lorenz via cfe-commits > wrote: >> Author: arphaman >> Date: Wed Jul 26 05:20:57 2017 >> New Revision: 309106 >> >> URL: http://llvm.org/viewvc/llvm-project?rev=309106&view=rev >> Log: >> Recommit r3083

r309186 - Revert r309106 "Recommit r308327 2nd time: Add a warning for missing"

2017-07-26 Thread Hans Wennborg via cfe-commits
Author: hans Date: Wed Jul 26 14:29:24 2017 New Revision: 309186 URL: http://llvm.org/viewvc/llvm-project?rev=309186&view=rev Log: Revert r309106 "Recommit r308327 2nd time: Add a warning for missing" The warning fires on non-suspicious code in Chromium. Reverting until a solution

Re: r309106 - Recommit r308327 2nd time: Add a warning for missing

2017-07-26 Thread Hans Wennborg via cfe-commits
On Wed, Jul 26, 2017 at 5:20 AM, Alex Lorenz via cfe-commits wrote: > Author: arphaman > Date: Wed Jul 26 05:20:57 2017 > New Revision: 309106 > > URL: http://llvm.org/viewvc/llvm-project?rev=309106&view=rev > Log: > Recommit r308327 2nd time: Add a warning for missing > '#pragma pack (pop)' and s

r309106 - Recommit r308327 2nd time: Add a warning for missing

2017-07-26 Thread Alex Lorenz via cfe-commits
Author: arphaman Date: Wed Jul 26 05:20:57 2017 New Revision: 309106 URL: http://llvm.org/viewvc/llvm-project?rev=309106&view=rev Log: Recommit r308327 2nd time: Add a warning for missing '#pragma pack (pop)' and suspicious uses of '#pragma pack' in included files The first recommit (r308441) cau