Re: [PATCH] D21823: [Driver] Add flags for enabling both types of PGO Instrumentation

2016-07-02 Thread Sean Silva via cfe-commits
On Sat, Jul 2, 2016 at 1:57 PM, Xinliang David Li wrote: > > > On Fri, Jul 1, 2016 at 4:32 PM, Sean Silva wrote: > >> silvas added inline comments. >> >> >> Comment at: lib/Driver/Tools.cpp:3560 >> @@ +3559,3 @@ >> +if

r274454 - [Refactor NFC] Rename the (non-CCE, fold-failure) Diag during constant expression evaluation as FFDiag.

2016-07-02 Thread Faisal Vali via cfe-commits
Author: faisalv Date: Sat Jul 2 17:34:24 2016 New Revision: 274454 URL: http://llvm.org/viewvc/llvm-project?rev=274454=rev Log: [Refactor NFC] Rename the (non-CCE, fold-failure) Diag during constant expression evaluation as FFDiag. Currently, we have CCEDiags (C++11 core constant expression

Re: [PATCH] D21823: [Driver] Add flags for enabling both types of PGO Instrumentation

2016-07-02 Thread Xinliang David Li via cfe-commits
On Fri, Jul 1, 2016 at 4:32 PM, Sean Silva wrote: > silvas added inline comments. > > > Comment at: lib/Driver/Tools.cpp:3560 > @@ +3559,3 @@ > +if (PGOTrainArg->getOption().matches(options::OPT_fpgo_train_EQ)) { > + if

Re: [PATCH] D20795: Added ASTStructure for analyzing the structure of Stmts.

2016-07-02 Thread Vassil Vassilev via cfe-commits
v.g.vassilev requested changes to this revision. This revision now requires changes to proceed. Comment at: include/clang/Basic/SourceManager.h:1173 @@ +1172,3 @@ + + // + A leftover from something else? Comment at:

r274442 - [X86][AVX512] Converted the MOVDDUP/MOVSLDUP/MOVSHDUP masked intrinsics to generic IR

2016-07-02 Thread Simon Pilgrim via cfe-commits
Author: rksimon Date: Sat Jul 2 12:16:25 2016 New Revision: 274442 URL: http://llvm.org/viewvc/llvm-project?rev=274442=rev Log: [X86][AVX512] Converted the MOVDDUP/MOVSLDUP/MOVSHDUP masked intrinsics to generic IR llvm companion patch imminent Modified:

Re: [PATCH] D21385: Adjust Registry interface to not require plugins to export a registry

2016-07-02 Thread Rudy Pons via cfe-commits
Ilod added a comment. The PrintFunctionNames plugin works fine, but not the SampleAnalyzerPlugin? The checker-plugins test fails. Not sure, but I suppose it's because the regitration method for checkers is different (a C method called clang_registerCheckers is retrieved in the DLL from the

r274433 - [CUDA] Move argument type lists to the stack. NFC.

2016-07-02 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sat Jul 2 07:03:57 2016 New Revision: 274433 URL: http://llvm.org/viewvc/llvm-project?rev=274433=rev Log: [CUDA] Move argument type lists to the stack. NFC. Modified: cfe/trunk/lib/CodeGen/CGCUDANV.cpp Modified: cfe/trunk/lib/CodeGen/CGCUDANV.cpp URL:

r274432 - Use arrays or initializer lists to feed ArrayRefs instead of SmallVector where possible.

2016-07-02 Thread Benjamin Kramer via cfe-commits
Author: d0k Date: Sat Jul 2 06:41:41 2016 New Revision: 274432 URL: http://llvm.org/viewvc/llvm-project?rev=274432=rev Log: Use arrays or initializer lists to feed ArrayRefs instead of SmallVector where possible. No functionality change intended Modified: