Re: [PATCH] D33467: Fix LLVM build errors if necent build of GCC 7 is used

2017-05-26 Thread Richard Smith via cfe-commits
This definitely looks like a GCC bug. On 26 May 2017 1:34 am, "David Abdurachmanov via Phabricator via cfe-commits" wrote: davidlt added a comment. This happens with recent GCC 7.1.1 (updated yesterday) and if compiled in C++1z mode. FAILED: /home/davidlt/root_patch_check/a/slc7_ppc64le_gcc7

Re: [PATCH] D33467: Fix LLVM build errors if necent build of GCC 7 is used

2017-05-26 Thread David Abdurachmanov via cfe-commits
Just reminder that this is in C++1z (C++17) mode, which I doubt is being widely used at this point. I was a change a few months ago that caused it, but cannot recall exact commit in GCC. david > On 26 May 2017, at 10:43, Chandler Carruth wrote: > > I kind of think this may be a GCC bug. It i

Re: [PATCH] D33467: Fix LLVM build errors if necent build of GCC 7 is used

2017-05-26 Thread Chandler Carruth via cfe-commits
Maybe Richard would be a better one to explain what on earth is going on here then... On Fri, May 26, 2017 at 1:51 AM David Abdurachmanov < david.abdurachma...@gmail.com> wrote: > Just reminder that this is in C++1z (C++17) mode, which I doubt is being > widely used at this point. > > I was a cha

Re: [PATCH] D33467: Fix LLVM build errors if necent build of GCC 7 is used

2017-05-26 Thread Chandler Carruth via cfe-commits
I kind of think this may be a GCC bug. It is quite strange that no one else has reported it, and that you're only seeing it in this context. We have many std::string flags and they are used in this way pretty commonly. I'm not seeing this with any other GCC-hosted build bot either, including a PPC