[clang] [lldb] [llvm] [openmp] [polly] fix(python): fix comparison to True/False (PR #91858)

2024-05-14 Thread Alex Langford via cfe-commits
https://github.com/bulbazord requested changes to this pull request. Hmm, actually, I'm not so sure about this change anymore. I went through PEP8 again and saw this: ``` Don’t compare boolean values to True or False using ==: # Correct: if greeting: # Wrong: if greeting == True: Worse: #

[clang] [lldb] [llvm] [openmp] [polly] fix(python): fix comparison to True/False (PR #91858)

2024-05-13 Thread Alex Langford via cfe-commits
https://github.com/bulbazord approved this pull request. LGTM The PR summary is empty, but it looks like the commit message has an explanation for this change, so that's fine. https://github.com/llvm/llvm-project/pull/91858 ___ cfe-commits mailing

[clang] [compiler-rt] [libc] [libclc] [libcxxabi] [lld] [lldb] [llvm] [mlir] Add clarifying parenthesis around non-trivial conditions in ternary expressions. (PR #90391)

2024-04-30 Thread Alex Langford via cfe-commits
bulbazord wrote: LLDB changes look fine. https://github.com/llvm/llvm-project/pull/90391 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [compiler-rt] [libc] [libclc] [libcxxabi] [lld] [lldb] [llvm] [mlir] llvm/lib/Target/X86/AsmParser/X86AsmParser.cpp:3804: lacking () for c… (PR #90391)

2024-04-29 Thread Alex Langford via cfe-commits
bulbazord wrote: > > Please update the PR subject as its a lot more than just X86AsmParser.cpp > > Hi @RKSimon > > All the issues mentioned are fixed. The title of the PR is misleading. The > title is the same as the issue (#85868) it corresponds to. Got a look to > other PR's and I thought

[clang] 266ec80 - [clang][DebugInfo] Respect fmodule-file-home-is-cwd in skeleton CUs for clang modules

2022-10-04 Thread Alex Langford via cfe-commits
Author: Alex Langford Date: 2022-10-04T11:25:43-07:00 New Revision: 266ec801fb23f9f5f1d61ca9466e0805fbdb78a7 URL: https://github.com/llvm/llvm-project/commit/266ec801fb23f9f5f1d61ca9466e0805fbdb78a7 DIFF: https://github.com/llvm/llvm-project/commit/266ec801fb23f9f5f1d61ca9466e0805fbdb78a7.diff

[clang] c57f034 - [clang][Sema] Add flag to LookupName to force C/ObjC codepath

2022-04-19 Thread Alex Langford via cfe-commits
Author: Alex Langford Date: 2022-04-19T12:57:14-07:00 New Revision: c57f03415f9668f942802ff7108410d57c2b10da URL: https://github.com/llvm/llvm-project/commit/c57f03415f9668f942802ff7108410d57c2b10da DIFF: https://github.com/llvm/llvm-project/commit/c57f03415f9668f942802ff7108410d57c2b10da.diff

[clang] b3f789e - [perf-training] Change profile file pattern string to use %4m instead of %p

2019-12-17 Thread Alex Langford via cfe-commits
Author: Xin-Xin Wang Date: 2019-12-17T12:12:21-08:00 New Revision: b3f789e037cbfdb1439c01a4eefc9ab9bb0d2c64 URL: https://github.com/llvm/llvm-project/commit/b3f789e037cbfdb1439c01a4eefc9ab9bb0d2c64 DIFF: https://github.com/llvm/llvm-project/commit/b3f789e037cbfdb1439c01a4eefc9ab9bb0d2c64.diff

r372668 - [NFCI] Return PathSensitiveBugReport where appropriate

2019-09-23 Thread Alex Langford via cfe-commits
Author: xiaobai Date: Mon Sep 23 15:24:47 2019 New Revision: 372668 URL: http://llvm.org/viewvc/llvm-project?rev=372668=rev Log: [NFCI] Return PathSensitiveBugReport where appropriate Some compilers have trouble converting unique_ptr to unique_ptr causing some functions to fail to compile.

r372363 - [NFCI] Always initialize const members of AttributeCommonInfo

2019-09-19 Thread Alex Langford via cfe-commits
Author: xiaobai Date: Thu Sep 19 17:16:32 2019 New Revision: 372363 URL: http://llvm.org/viewvc/llvm-project?rev=372363=rev Log: [NFCI] Always initialize const members of AttributeCommonInfo Some compilers require that const fields of an object must be explicitly initialized by the constructor.

r368950 - [NFCI] Always initialize BugReport const fields

2019-08-14 Thread Alex Langford via cfe-commits
Author: xiaobai Date: Wed Aug 14 17:58:51 2019 New Revision: 368950 URL: http://llvm.org/viewvc/llvm-project?rev=368950=rev Log: [NFCI] Always initialize BugReport const fields Summary: Some compilers require that const fields of an object must be explicitly initialized by the constructor. I ran

r368779 - [analyzer] Don't delete TaintConfig copy constructor

2019-08-14 Thread Alex Langford via cfe-commits
Author: xiaobai Date: Tue Aug 13 18:09:07 2019 New Revision: 368779 URL: http://llvm.org/viewvc/llvm-project?rev=368779=rev Log: [analyzer] Don't delete TaintConfig copy constructor Summary: Explicitly deleting the copy constructor makes compiling the function `ento::registerGenericTaintChecker`