[clang] 83973cf - [clang][modules][deps] Including module maps are affecting

2022-12-01 Thread Jan Svoboda via cfe-commits
Author: Jan Svoboda Date: 2022-12-01T20:16:17-08:00 New Revision: 83973cf157f7850eb133a4bbfa0f8b7958bad215 URL: https://github.com/llvm/llvm-project/commit/83973cf157f7850eb133a4bbfa0f8b7958bad215 DIFF: https://github.com/llvm/llvm-project/commit/83973cf157f7850eb133a4bbfa0f8b7958bad215.diff L

[PATCH] D137206: [clang][modules][deps] Including module maps are affecting

2022-12-01 Thread Jan Svoboda via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG83973cf157f7: [clang][modules][deps] Including module maps are affecting (authored by jansvoboda11). Changed prior to commit: https://reviews.llvm

[PATCH] D139148: Fix nullptr dereference found by Coverity static analysis tool

2022-12-01 Thread Sindhu Chittireddy via Phabricator via cfe-commits
schittir added a comment. In D139148#3965404 , @shafik wrote: > This looks like the correct thing to do but I see further down there are also > unconditional uses of `Initializer` that should also be guarded with a check. > It would be good to fix those

[PATCH] D138702: support for HIP non hostcall printf

2022-12-01 Thread Vikram Hegde via Phabricator via cfe-commits
vikramRH added a comment. There were some issues reported where it was found that pcie atomics are not available in some environments, as a result hostcall and HIP-Printf were also not available. This Patch is intended to provide an alternative in such scenarios while hostcall based implementat

[PATCH] D85802: [clang] Add -fc++-abi= flag for specifying which C++ ABI to use

2022-12-01 Thread David Blaikie via Phabricator via cfe-commits
dblaikie added a comment. So, some context for how this came up/what I'm trying to address: So I started here: D117616 (for the googlers playing at home, see: b/198958855 for the proximal motivation) - not packing non-POD members of packed structs. That lead t

[clang] 8ab388e - [clang][deps] Convert file dependency paths to the native style

2022-12-01 Thread Jan Svoboda via cfe-commits
Author: Jan Svoboda Date: 2022-12-01T21:07:56-08:00 New Revision: 8ab388e158528d9af5eb0376ef698b243d946f19 URL: https://github.com/llvm/llvm-project/commit/8ab388e158528d9af5eb0376ef698b243d946f19 DIFF: https://github.com/llvm/llvm-project/commit/8ab388e158528d9af5eb0376ef698b243d946f19.diff L

[PATCH] D139168: [C++20] [Modules] [ClangScanDeps] Enable to print make-style dependency file within P1689 format (4/4)

2022-12-01 Thread Chuanqi Xu via Phabricator via cfe-commits
ChuanqiXu created this revision. ChuanqiXu added reviewers: ben.boeckel, Bigcheese, jansvoboda11. Herald added a project: All. ChuanqiXu requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Required in https://reviews.llvm.org/D137534. The buil

[PATCH] D137534: [C++20] [Modules] [ClangScanDeps] Allow clang-scan-deps to without specified compilation database in P1689 (3/3)

2022-12-01 Thread Chuanqi Xu via Phabricator via cfe-commits
ChuanqiXu added a comment. In D137534#3965012 , @ben.boeckel wrote: > So a few things I see when trying to put this into CMake: > > - P1689 output is only to `stdout`? > - Is there a way to get `-MF` output for the files

[clang] f0ce827 - Revert "[clang][modules][deps] Including module maps are affecting"

2022-12-01 Thread Jan Svoboda via cfe-commits
Author: Jan Svoboda Date: 2022-12-01T21:39:51-08:00 New Revision: f0ce827c6972009c5052f8447c6aadf4e4be6113 URL: https://github.com/llvm/llvm-project/commit/f0ce827c6972009c5052f8447c6aadf4e4be6113 DIFF: https://github.com/llvm/llvm-project/commit/f0ce827c6972009c5052f8447c6aadf4e4be6113.diff L

[clang] 67f3405 - Revert "[clang][modules][deps] Parent module maps are affecting"

2022-12-01 Thread Jan Svoboda via cfe-commits
Author: Jan Svoboda Date: 2022-12-01T21:39:51-08:00 New Revision: 67f34054d6ea8e40fd10cb74441d5ccab004c75b URL: https://github.com/llvm/llvm-project/commit/67f34054d6ea8e40fd10cb74441d5ccab004c75b DIFF: https://github.com/llvm/llvm-project/commit/67f34054d6ea8e40fd10cb74441d5ccab004c75b.diff L

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-01 Thread Freddy, Ye via Phabricator via cfe-commits
FreddyYe created this revision. Herald added a project: All. FreddyYe requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D139170 Files: clang/lib/Basic/Targets/X86.h clang/te

[PATCH] D136815: [clang][Interp] Unify visiting variable declarations

2022-12-01 Thread Timm Bäder via Phabricator via cfe-commits
tbaeder updated this revision to Diff 479526. tbaeder added a comment. Meh, just use an assert :| CHANGES SINCE LAST ACTION https://reviews.llvm.org/D136815/new/ https://reviews.llvm.org/D136815 Files: clang/lib/AST/Interp/ByteCodeExprGen.cpp clang/lib/AST/Interp/ByteCodeExprGen.h clan

[PATCH] D139171: Don't revisit the subexpressions of PseudoObjectExpr when building a ParentMap

2022-12-01 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak created this revision. ahatanak added reviewers: rjmccall, jordan_rose. ahatanak added a project: clang. Herald added a project: All. ahatanak requested review of this revision. The assertion that is removed in this patch was failing when ObjC dot notation expressions appear in both side

[PATCH] D139172: [clang] Mark CWG554 as N/A

2022-12-01 Thread Vlad Serebrennikov via Phabricator via cfe-commits
Endill created this revision. Endill added a reviewer: clang-language-wg. Herald added a project: All. Endill requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. P1787 : //CWG554 is resolved by using the word “s

[PATCH] D139170: [X86][clang] Lift _BitInt() supported max width.

2022-12-01 Thread Matthias Gehre via Phabricator via cfe-commits
mgehre-amd added a comment. Do other targets not support > 128 bit integers, or is this PR only the first conservative step of lifting the limit? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D139170/new/ https://reviews.llvm.org/D139170 _

<    1   2   3