[PATCH] D105759: [WIP] Implement P2361 Unevaluated string literals

erichkeane added inline comments. Comment at: clang/lib/Lex/LiteralSupport.cpp:1821-1823 +<< UDSuffixBuf << UDSuffix +<< SourceRange(UDSuffixTokLoc, UDSuffixTokLoc) +<< SourceRange(TokLoc, TokLoc); cor3ntin wrote: >

[PATCH] D109632: [clang] de-duplicate methods from AST files

rmaz added a comment. In D109632#3022209 , @vsapsai wrote: > That patch looks correct, I was experimenting with exactly the same local > change. Have you tried D110123 on your > original build? In my testing with synth

[PATCH] D105191: [Clang][OpenMP] Add partial support for Static Device Libraries

saiislam updated this revision to Diff 375648. saiislam added a comment. fixed the typos in the test cases Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105191/new/ https://reviews.llvm.org/D105191 Files: clang/lib/Driver/ToolChains/AMDGPUOpenMP

[PATCH] D110641: Implement P0857R0 -Part B: requires clause for template-template params

aaron.ballman added a reviewer: aaron.ballman. aaron.ballman added inline comments. Comment at: clang/lib/Parse/ParseTemplate.cpp:876 +/// In C++20: +/// template-head: [C++ temp.pre] +/// template '<' template-parameter-list '>' requires-clause[opt] ---

[PATCH] D110641: Implement P0857R0 -Part B: requires clause for template-template params

aaron.ballman added a comment. Btw, the CI failure for `x64 debian > Clang.SemaTemplate::concepts.cpp` looks relevant. Not certain why I don't see a similar failure for Windows though. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110641/new/ https://reviews.llvm.org/D110641 _

RE: [PATCH] D102107: [OpenMP] Codegen aggregate for outlined function captures

[AMD Official Use Only] Hi George, I will do it now ... -Original Message- From: Giorgis Georgakoudis via Phabricator Sent: Tuesday, September 28, 2021 1:30 PM To: georgakoud...@llnl.gov; jdoerf...@anl.gov; jhub...@vols.utk.edu Cc: llvm-comm...@lists.llvm.org; a335p...@edu.uwaterloo.ca

[PATCH] D109967: Simplify handling of builtin with inline redefinition

kda added a comment. Broke fast buildbot: https://lab.llvm.org/buildbot/#/builders/5/builds/12360 reverting... A guide to reproducing sanitizer bots can be found here: https://github.com/google/sanitizers/wiki/SanitizerBotReproduceBuild Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[clang] fd582ee - [CUDA] Move CUDA SDK include path further down the include search path.

Author: Artem Belevich Date: 2021-09-28T11:29:28-07:00 New Revision: fd582eeffe582665eacac522617a15e17e9872cd URL: https://github.com/llvm/llvm-project/commit/fd582eeffe582665eacac522617a15e17e9872cd DIFF: https://github.com/llvm/llvm-project/commit/fd582eeffe582665eacac522617a15e17e9872cd.diff

[PATCH] D110596: [CUDA] Move CUDA SDK include path further down the include search path.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGfd582eeffe58: [CUDA] Move CUDA SDK include path further down the include search path. (authored by tra). Repository: rG LLVM Github Monorepo CHAN

[PATCH] D110364: [clang] Rework dontcall attributes

aeubanks created this revision. Herald added subscribers: ormris, dexonsmith, jdoerfert, pengfei, steven_wu, hiraditya. aeubanks updated this revision to Diff 374663. aeubanks added a comment. aeubanks added reviewers: dblaikie, nickdesaulniers. aeubanks published this revision for review. Herald

[clang] 0d76d48 - Revert "Simplify handling of builtin with inline redefinition"

Author: Kevin Athey Date: 2021-09-28T11:30:37-07:00 New Revision: 0d76d4833dd2815e0b1c786250f474d222f6a0a1 URL: https://github.com/llvm/llvm-project/commit/0d76d4833dd2815e0b1c786250f474d222f6a0a1 DIFF: https://github.com/llvm/llvm-project/commit/0d76d4833dd2815e0b1c786250f474d222f6a0a1.diff L

[PATCH] D110641: Implement P0857R0 -Part B: requires clause for template-template params

erichkeane added inline comments. Comment at: clang/lib/Parse/ParseTemplate.cpp:876 +/// In C++20: +/// template-head: [C++ temp.pre] +/// template '<' template-parameter-list '>' requires-clause[opt] aaron.ballman wrote: > Rather than duplicate the

[clang] c6edf26 - [docs ]Fix indentation

Author: Akira Hatanaka Date: 2021-09-28T11:31:39-07:00 New Revision: c6edf26f9a283564a5697068c845a8ff03386086 URL: https://github.com/llvm/llvm-project/commit/c6edf26f9a283564a5697068c845a8ff03386086 DIFF: https://github.com/llvm/llvm-project/commit/c6edf26f9a283564a5697068c845a8ff03386086.diff

[clang-tools-extra] 9b944c1 - Revert "Add support for `NOLINTBEGIN` ... `NOLINTEND` comments"

Author: Aaron Ballman Date: 2021-09-28T14:49:27-04:00 New Revision: 9b944c184396ce55a3ad608779cc326ba12c9ee3 URL: https://github.com/llvm/llvm-project/commit/9b944c184396ce55a3ad608779cc326ba12c9ee3 DIFF: https://github.com/llvm/llvm-project/commit/9b944c184396ce55a3ad608779cc326ba12c9ee3.diff

[PATCH] D108560: [clang-tidy] Add support for `NOLINTBEGIN` ... `NOLINTEND` comments

aaron.ballman added a comment. I had to revert in 9b944c184396ce55a3ad608779cc326ba12c9ee3 because there are testing failures. FAIL: Clang Tools :: clang-tidy/infrastructure/nolintbeginend.cpp (1 of 1) ***

[PATCH] D110364: [clang] Rework dontcall attributes

nickdesaulniers accepted this revision. nickdesaulniers added a comment. This revision is now accepted and ready to land. thanks for the patch! Comment at: llvm/lib/IR/DiagnosticInfo.cpp:408-427 + if (F->hasFnAttribute("dontcall-error")) { +unsigned LocCookie = 0; +aut

[PATCH] D105759: [WIP] Implement P2361 Unevaluated string literals

aaron.ballman added inline comments. Comment at: clang/lib/Lex/LiteralSupport.cpp:1821-1823 +<< UDSuffixBuf << UDSuffix +<< SourceRange(UDSuffixTokLoc, UDSuffixTokLoc) +<< SourceRange(TokLoc, TokLoc); erichkeane wro

[clang] c3717b6 - Simplify handling of builtin with inline redefinition

Author: serge-sans-paille Date: 2021-09-28T21:00:47+02:00 New Revision: c3717b6858d32d64514a187ede1a77be8ba4e542 URL: https://github.com/llvm/llvm-project/commit/c3717b6858d32d64514a187ede1a77be8ba4e542 DIFF: https://github.com/llvm/llvm-project/commit/c3717b6858d32d64514a187ede1a77be8ba4e542.d

[PATCH] D109967: Simplify handling of builtin with inline redefinition

serge-sans-paille added a comment. @kda : thanks for the builbot reference. The issue you're pointing at occurs *before* bd379915de38a9af3d65e19075a6a64ebbb8d6db which specifically fixes the issue spotted by the buildbot. A

[PATCH] D109632: [clang] de-duplicate methods from AST files

rmaz added a comment. Avoiding serializing external methods in `ASTMethodPoolTrait` is working successfully, although the times are not as good as using set deduplication: | **file** | **baseline** | **set dedupe** | **no external** | | Total | 149.68 | 92.97 | 109.83

[PATCH] D110641: Implement P0857R0 -Part B: requires clause for template-template params

erichkeane added a comment. In D110641#3028173 , @aaron.ballman wrote: > Btw, the CI failure for `x64 debian > Clang.SemaTemplate::concepts.cpp` looks > relevant. Not certain why I don't see a similar failure for Windows though. Looks like I miseed up

[PATCH] D108301: [MSP430][Clang] Update hard-coded MCU data

jozefl updated this revision to Diff 375666. jozefl marked 5 inline comments as done. jozefl added a comment. Thanks, good points, I have addressed them in this updated patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108301/new/ https://review

[PATCH] D109902: [PowerPC] Improved codegen related to xscvdpsxws/xscvdpuxws

Conanap updated this revision to Diff 375667. Conanap added a comment. Added context, changed var names in aptterns CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109902/new/ https://reviews.llvm.org/D109902 Files: llvm/lib/Target/PowerPC/PPCInstrVSX.td llvm/test/CodeGen/PowerPC/tes

[PATCH] D109902: [PowerPC] Improved codegen related to xscvdpsxws/xscvdpuxws

Conanap marked 2 inline comments as done. Conanap added inline comments. Comment at: llvm/lib/Target/PowerPC/PPCInstrVSX.td:4150 +0))>; +def : Pat<(v4i32 (insertelt v4i32:$VEC, DblToUInt.A, 0)), + (v4i32 (XXINSERTW v4i32:$VEC,

[PATCH] D110641: Implement P0857R0 -Part B: requires clause for template-template params

erichkeane added a comment. Ok, Aaron and I are pretty confused here I think (based on offline discussions), so I'm hoping for some guidance. `ParseTemplateTemplateParameter` does all the work to parse a template-template parameter, then calls `ActOnTemplateTemplateParameter`, which handles add

[PATCH] D109967: Simplify handling of builtin with inline redefinition

thakis added a comment. This breaks tests on windows and macOS eg http://45.33.8.238/win/46075/step_7.txt Please take a look and revert for now if it takes a while to fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109967/new/ https://reviews

[PATCH] D109967: Simplify handling of builtin with inline redefinition

thakis added a comment. For macOS, see eg http://45.33.8.238/macm1/18806/step_7.txt Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109967/new/ https://reviews.llvm.org/D109967 ___ cfe-commits mailing list

[PATCH] D102107: [OpenMP] Codegen aggregate for outlined function captures

ronlieb added a comment. i backed up to your reverted patch, and applied this one. I see some new errors libomptarget :: amdgcn-amd-amdhsa :: mapping/declare_mapper_target.cpp libomptarget :: amdgcn-amd-amdhsa :: mapping/declare_mapper_target_data.cpp libomptarget :: amdgcn-amd-amdhsa :: m

[PATCH] D109703: [DebugInfo] Fix scope for local static variables

ellis added a comment. In D109703#3018935 , @krisb wrote: > @dblaikie yeah, the problem(s) seemed easier and smaller :( > > Basically, we have two issues with local scopes here: > (1) function-scoped entities like static variables, type > definitions/typ

[PATCH] D110625: [analyzer] canonicalize special case of structure/pointer deref

martong added a comment. Thanks Vince! Nice work! Do you think it would be worth to have a test that checks the equality of a double pointer and a bi-dimensional array? Something like: void struct_pointer_canon(struct s **ps) { struct s ss = **ps; clang_analyzer_eval(&(ps[0][0].v) ==

[PATCH] D105191: [Clang][OpenMP] Add partial support for Static Device Libraries

saiislam updated this revision to Diff 375672. saiislam added a comment. fixed nvptx test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105191/new/ https://reviews.llvm.org/D105191 Files: clang/lib/Driver/ToolChains/AMDGPUOpenMP.cpp clang/lib/

[clang] 1ecb1bc - Fix memcpy-nobuiltin.c test case

Author: serge-sans-paille Date: 2021-09-28T21:55:42+02:00 New Revision: 1ecb1bc3e214c8da53a7fda14f428786441109d7 URL: https://github.com/llvm/llvm-project/commit/1ecb1bc3e214c8da53a7fda14f428786441109d7 DIFF: https://github.com/llvm/llvm-project/commit/1ecb1bc3e214c8da53a7fda14f428786441109d7.d

[PATCH] D109967: Simplify handling of builtin with inline redefinition

serge-sans-paille added a comment. @thakis if rG1ecb1bc3e214 doesn't work, I'll revert. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109967/new/ https://reviews.llvm.org/D1099

[PATCH] D109967: Simplify handling of builtin with inline redefinition

serge-sans-paille added a comment. @thakis: fine on OSX: http://45.33.8.238/macm1/18808/summary.html Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109967/new/ https://reviews.llvm.org/D109967 ___ cfe-com

[PATCH] D110641: Implement P0857R0 -Part B: requires clause for template-template params

erichkeane updated this revision to Diff 375680. erichkeane added a comment. Moments after my last comment, i figured out the scope issue, which makes the names in the template-template-parameter parameter-list in scope for this requires clause. I don't think it makes SENSE that this is the cas

[PATCH] D105191: [Clang][OpenMP] Add partial support for Static Device Libraries

saiislam added a comment. Ping. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105191/new/ https://reviews.llvm.org/D105191 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/c

[PATCH] D110422: [AIX] Enable PGO without LTO

MaskRay added inline comments. Comment at: llvm/lib/Transforms/Instrumentation/InstrProfiling.cpp:865 + // Due to the current limitation of binder, the duplicate weak symbols in the + // same csect won't be discarded. When there are duplicate weak symbols, "D

[PATCH] D106102: [analyzer][solver] Introduce reasoning for not equal to operator

manas updated this revision to Diff 375682. manas added a comment. Move method to a specialized template for VisitBinaryOperator Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106102/new/ https://reviews.llvm.org/D106102 Files: clang/lib/StaticAn

[PATCH] D109967: Simplify handling of builtin with inline redefinition

serge-sans-paille added a comment. And on Windows: http://45.33.8.238/win/46077/summary.html ! Thanks for pointing those. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109967/new/ https://reviews.llvm.org/D109967 _

[PATCH] D110481: fixes bug #51926 where dangling comma caused overrun

HazardyKnusperkeks accepted this revision. HazardyKnusperkeks added a comment. This revision is now accepted and ready to land. In D110481#3027289 , @feg208 wrote: > By one other bug I mean an entirely separate bug that I will handle in a > separate PR.

[PATCH] D110422: [AIX] Enable PGO without LTO

jsji updated this revision to Diff 375688. jsji marked an inline comment as done. jsji added a comment. Address comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110422/new/ https://reviews.llvm.org/D110422 Files: clang/lib/Driver/ToolChai

[PATCH] D106102: [analyzer][solver] Introduce reasoning for not equal to operator

manas updated this revision to Diff 375689. manas added a comment. Rebase Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106102/new/ https://reviews.llvm.org/D106102 Files: clang/lib/StaticAnalyzer/Core/RangeConstraintManager.cpp clang/test/Ana

[PATCH] D108560: [clang-tidy] Add support for `NOLINTBEGIN` ... `NOLINTEND` comments

salman-javed-nz updated this revision to Diff 375685. salman-javed-nz added a comment. Resolving build error due to failed unit test. On some build bots, the clang-tidy diagnostics coming from `error_in_include.inc` get printed BEFORE all other diagnostics, REGARDLESS of the location of the `#i

[PATCH] D110485: Support [[no_unique_address]] for all targets.

expnkx updated this revision to Diff 375690. expnkx added a comment. I cannot find out where to add attribute that starts with msvc::x. I am just applying this "potential" patch to the clang upstream. In the future we can just remove TargetItaniumCXXABI attr in the clang table gen file witho

[PATCH] D110653: [PowerPC] The builtins load8r and store8r are Power 7 plus.

stefanp created this revision. stefanp added reviewers: nemanjai, lei. Herald added subscribers: shchenz, kbarton, hiraditya. stefanp requested review of this revision. Herald added projects: clang, LLVM. Herald added a subscriber: cfe-commits. This patch makes sure that the builtins __builtin_ppc

[PATCH] D106102: [analyzer][solver] Introduce reasoning for not equal to operator

manas added a comment. The pre-merge checks fail due to the patch being unable to get applied. The troubleshooting suggest to update the patch via `arc diff ```git merge-base HEAD origin``` --update D106102` an

[PATCH] D110481: fixes bug #51926 where dangling comma caused overrun

feg208 added a comment. Absolutely in agreement Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110481/new/ https://reviews.llvm.org/D110481 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://

[PATCH] D110422: [AIX] Enable PGO without LTO

MaskRay added a comment. I still think the description can be rephrased to be shorter and clearer, and am not very sure this workaround should be added. But I will take a vacation and be back after one week and don't want to appear that I am blocking this change. So LGTM once you decrease the n

[PATCH] D110655: [OpenMP] Apply OpenMP assumptions to applicable call sites

jhuber6 created this revision. jhuber6 added a reviewer: jdoerfert. Herald added subscribers: guansong, yaxunl. jhuber6 requested review of this revision. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. This patch adds OpenMP assumption attributes to call sites in a

[PATCH] D108560: [clang-tidy] Add support for `NOLINTBEGIN` ... `NOLINTEND` comments

salman-javed-nz added a comment. F19296441: nolintbeginend.cpp.tmp.cpp.msg You can see in the attached file, that when I ran the unit test on a x86_64 Windows 10 PC, the diagnostic from `error_in_include.inc` is printed at the end. However, on this this clang

[PATCH] D110364: [clang] Rework dontcall attributes

aeubanks updated this revision to Diff 375698. aeubanks added a comment. DRY Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110364/new/ https://reviews.llvm.org/D110364 Files: clang/lib/CodeGen/CodeGenAction.cpp clang/lib/CodeGen/CodeGenModule.

[PATCH] D110656: [clang][Sema] Warn on uninitialized array elments

beanz created this revision. beanz added reviewers: aaron.ballman, rsmith, RKSimon, dexonsmith. beanz requested review of this revision. Herald added a project: clang. When defining a statically sized array with explicit array initializers, having a warning for uninitialized members is nice. This

[clang] 2943071 - [clang] Rework dontcall attributes

Author: Arthur Eubanks Date: 2021-09-28T14:21:10-07:00 New Revision: 2943071e2ee0c7f31f34062a44d12aeb0e3a66fd URL: https://github.com/llvm/llvm-project/commit/2943071e2ee0c7f31f34062a44d12aeb0e3a66fd DIFF: https://github.com/llvm/llvm-project/commit/2943071e2ee0c7f31f34062a44d12aeb0e3a66fd.diff

[PATCH] D110364: [clang] Rework dontcall attributes

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2943071e2ee0: [clang] Rework dontcall attributes (authored by aeubanks). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://

[clang] 952f030 - [test] Pin some RUN lines in optimization-remark.c to new PM

Author: Arthur Eubanks Date: 2021-09-28T14:29:33-07:00 New Revision: 952f030fe6ade193ead8f23a7654cf8d2c7aa3df URL: https://github.com/llvm/llvm-project/commit/952f030fe6ade193ead8f23a7654cf8d2c7aa3df DIFF: https://github.com/llvm/llvm-project/commit/952f030fe6ade193ead8f23a7654cf8d2c7aa3df.diff

[clang] b69a2c8 - Revert "[test] Pin some RUN lines in optimization-remark.c to new PM"

Author: Arthur Eubanks Date: 2021-09-28T14:42:37-07:00 New Revision: b69a2c8eeca1fee26283ca4c2f99c787aabba067 URL: https://github.com/llvm/llvm-project/commit/b69a2c8eeca1fee26283ca4c2f99c787aabba067 DIFF: https://github.com/llvm/llvm-project/commit/b69a2c8eeca1fee26283ca4c2f99c787aabba067.diff

[PATCH] D110280: [modules] Fix IRGen assertion on accessing ObjC ivar inside a method.

vsapsai added a comment. In D110280#3016911 , @rjmccall wrote: > Hmm. It sounds like we misbehave if we're working with a non-canonical ivar. > While it does seem preferable in general for lookups done after merging > modules to return the canonical i

[clang] 7833d20 - Revert "[clang] Rework dontcall attributes"

Author: Arthur Eubanks Date: 2021-09-28T14:49:27-07:00 New Revision: 7833d20f1fd575fac89ce76822ffd561a40552e5 URL: https://github.com/llvm/llvm-project/commit/7833d20f1fd575fac89ce76822ffd561a40552e5 DIFF: https://github.com/llvm/llvm-project/commit/7833d20f1fd575fac89ce76822ffd561a40552e5.diff

[PATCH] D110422: [AIX] Change the linkage of profiling counter/data to be private

jsji updated this revision to Diff 375709. jsji added a comment. Split the Driver changes to another patch, also fixed the duplicate code. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110422/new/ https://reviews.llvm.org/D110422 Files: clang/te

[PATCH] D110656: [clang][Sema] Warn on uninitialized array elments

beanz updated this revision to Diff 375711. beanz added a comment. Updating so that the warning doesn't fire on an empty initializer list. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110656/new/ https://reviews.llvm.org/D110656 Files: clang/in

[PATCH] D109625: [compiler-rt] Ensure required deps for tests targets are actually built

leonardchan updated this revision to Diff 375713. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D109625/new/ https://reviews.llvm.org/D109625 Files: compiler-rt/test/CMakeLists.txt Index: compiler-rt/test/CMakeLists.txt ==

[PATCH] D110422: [AIX] Change the linkage of profiling counter/data to be private

jsji updated this revision to Diff 375716. jsji added a comment. Remove empty line. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110422/new/ https://reviews.llvm.org/D110422 Files: clang/test/Profile/cxx-templates.cpp llvm/lib/Transforms/Inst

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path

MaskRay created this revision. MaskRay added reviewers: phosek, sylvestre.ledru. Herald added a subscriber: pengfei. MaskRay requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This is needed when config.guess is removed (D109837

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path

sylvestre.ledru added a comment. just to make sure I understand, it will also update the install or "just" the dumpmachine option ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110663/new/ https://reviews.llvm.org/D110663 ___

[PATCH] D110485: Support [[no_unique_address]] for all targets.

expnkx updated this revision to Diff 375720. expnkx added a comment. it should first update the field and then test no_unique_address CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110485/new/ https://reviews.llvm.org/D110485 Files: clang/lib/AST/RecordLayoutBuilder.cpp Index: clang

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path

MaskRay updated this revision to Diff 375722. MaskRay added a comment. add missing file Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110663/new/ https://reviews.llvm.org/D110663 Files: clang/lib/Driver/ToolChain.cpp clang/test/Driver/Inputs/

[PATCH] D110422: [AIX] Change the linkage of profiling counter/data to be private

jsji added a comment. @MaskRay Thank you so much for your review and discussion! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110422/new/ https://reviews.llvm.org/D110422 ___ cfe-commits mailing list cf

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path

MaskRay added a comment. In D110663#3028811 , @sylvestre.ledru wrote: > just to make sure I understand, it will also update the install or "just" the > dumpmachine option ? If `LLVM_DEFAULT_TARGET_TRIPLE` is `x86_64-linux-gnu, `clang -dumpmachine` will

[clang] aa53785 - Reland [clang] Rework dontcall attributes

Author: Arthur Eubanks Date: 2021-09-28T15:31:30-07:00 New Revision: aa53785f23b2b89a9a423af131697b1f7c92869f URL: https://github.com/llvm/llvm-project/commit/aa53785f23b2b89a9a423af131697b1f7c92869f DIFF: https://github.com/llvm/llvm-project/commit/aa53785f23b2b89a9a423af131697b1f7c92869f.diff

[PATCH] D110625: [analyzer] canonicalize special case of structure/pointer deref

vabridgers added a comment. @martong, Thanks for yes for sure! I'll post an update soon with a few minor test improvements. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110625/new/ https://reviews.llvm.org/D110625 ___

[PATCH] D110625: [analyzer] canonicalize special case of structure/pointer deref

vabridgers updated this revision to Diff 375727. vabridgers added a comment. improve test cases per @martong Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110625/new/ https://reviews.llvm.org/D110625 Files: clang/lib/StaticAnalyzer/Core/Store.cp

[clang] a36227c - fixes bug #51926 where dangling comma caused overrun

Author: Fred Grim Date: 2021-09-28T15:59:37-07:00 New Revision: a36227cb2b6a14fc30bfd303328d1d1abb632010 URL: https://github.com/llvm/llvm-project/commit/a36227cb2b6a14fc30bfd303328d1d1abb632010 DIFF: https://github.com/llvm/llvm-project/commit/a36227cb2b6a14fc30bfd303328d1d1abb632010.diff LOG

[PATCH] D110481: fixes bug #51926 where dangling comma caused overrun

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa36227cb2b6a: fixes bug #51926 where dangling comma caused overrun (authored by feg208). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST A

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path

smeenai added inline comments. Comment at: clang/lib/Driver/ToolChain.cpp:494 + std::string Ret(P); + // When LLVM_DEFAULT_TARGET_TRIPLE uses Debian multiarch style + // "x86_64-linux-gnu" (no vendor part), use the unnormalized This is effectively reverting th

[PATCH] D110665: [clang] Don't use the AST to display backend diagnostics

aeubanks created this revision. aeubanks added reviewers: dblaikie, rnk. aeubanks requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. We keep a map from function name to source location so we don't have to do it via looking up a source location

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path

lebedev.ri added a comment. This doesn't help the world-breaking D107799 on debian for me. I don't know if this stands on it's own. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110663/new/ https://reviews.llvm.o

[PATCH] D109701: [clang] Emit SARIF Diagnostics: Create `clang::SarifDocumentWriter` interface

vaibhav.y updated this revision to Diff 375734. vaibhav.y marked 34 inline comments as done. vaibhav.y added a comment. Address comments from upstream review: - Mark anonymous functions static, improve documentation - Coding style fixes - Use Ref types as they are intended (as non-owning lightwei

[PATCH] D109701: [clang] Emit SARIF Diagnostics: Create `clang::SarifDocumentWriter` interface

vaibhav.y added inline comments. Comment at: clang/include/clang/Basic/Sarif.h:95 +// +/// Since every in clang artifact MUST have a location (there being no nested +/// artifacts), the creation method \ref SarifArtifact::create requires a aaron.ballman wrote: >

[PATCH] D110668: [clang-cl] Accept `#pragma warning(disable : N)` for some N

thakis created this revision. thakis added a reviewer: hans. thakis added a project: clang. Herald added subscribers: dexonsmith, dang, mgorny. thakis requested review of this revision. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. clang-cl maps /wd to -Wno-flags for a

[PATCH] D105168: [RISCV] Unify the arch string parsing logic to RISCVISAInfo.

kito-cheng added a comment. Herald added a subscriber: achieveartificialintelligence. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D105168/new/ https://reviews.llvm.org/D105168 ___ cfe-commits maili

[PATCH] D110635: [clang] Let PPCallbacks::PragmaWarning() pass specifier as enum instead of string

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5cf060614097: [clang] Let PPCallbacks::PragmaWarning() pass specifier as enum instead of… (authored by thakis). Repository: rG LLVM Github Monorep

[clang-tools-extra] 5cf0606 - [clang] Let PPCallbacks::PragmaWarning() pass specifier as enum instead of string

Author: Nico Weber Date: 2021-09-28T19:47:27-04:00 New Revision: 5cf0606140974b7560490778725c134eb35f0cc7 URL: https://github.com/llvm/llvm-project/commit/5cf0606140974b7560490778725c134eb35f0cc7 DIFF: https://github.com/llvm/llvm-project/commit/5cf0606140974b7560490778725c134eb35f0cc7.diff LO

[PATCH] D106550: [PowerPC] Allow MMA built-ins to accept restrict and volatile qualified pointers

nemanjai accepted this revision. nemanjai added a comment. This revision is now accepted and ready to land. LGTM. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D106550/new/ https://reviews.llvm.org/D106550 __

[PATCH] D110364: [clang] Rework dontcall attributes

thakis added a comment. Looks like this breaks tests on windows: http://45.33.8.238/win/46089/step_7.txt Please take a look, and revert for now if it takes a while to fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110364/new/ https://reviews.l

[PATCH] D109902: [PowerPC] Improved codegen related to xscvdpsxws/xscvdpuxws

nemanjai requested changes to this revision. nemanjai added a comment. This revision now requires changes to proceed. I really don't understand what happened now. It seems that you have simply reverted to an older version of this patch. The test case appears to not have been pre-committed any lo

[clang] 2d56fbf - [test] Specify triple in backend-attribute-error-warning.cpp

Author: Arthur Eubanks Date: 2021-09-28T17:00:10-07:00 New Revision: 2d56fbf6cd16bb7d2ae6c726b37d2096b9bdcd5a URL: https://github.com/llvm/llvm-project/commit/2d56fbf6cd16bb7d2ae6c726b37d2096b9bdcd5a DIFF: https://github.com/llvm/llvm-project/commit/2d56fbf6cd16bb7d2ae6c726b37d2096b9bdcd5a.diff

[PATCH] D110364: [clang] Rework dontcall attributes

aeubanks added a comment. In D110364#3028995 , @thakis wrote: > Looks like this breaks tests on windows: > http://45.33.8.238/win/46089/step_7.txt > > Please take a look, and revert for now if it takes a while to fix. Should be fixed with 2d56fbf6cd16bb

[clang] d87bdc2 - Clean up large copies of binaries copied into temp directories in tests

Author: Teresa Johnson Date: 2021-09-28T17:04:09-07:00 New Revision: d87bdc272ba47b7d9109ff5c7191454ab2ae6fcb URL: https://github.com/llvm/llvm-project/commit/d87bdc272ba47b7d9109ff5c7191454ab2ae6fcb DIFF: https://github.com/llvm/llvm-project/commit/d87bdc272ba47b7d9109ff5c7191454ab2ae6fcb.diff

[PATCH] D110276: Clean up large copies of binaries copied into temp directories in tests

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGd87bdc272ba4: Clean up large copies of binaries copied into temp directories in tests (authored by tejohnson). Repository: rG LLVM Github Monorepo

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path

jrtc27 added a comment. NB: summary has x86_64-known rather than x86_64-unknown in a couple of places Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110663/new/ https://reviews.llvm.org/D110663 ___ cfe-co

[PATCH] D102107: [OpenMP] Codegen aggregate for outlined function captures

ggeorgakoudis added a comment. In D102107#3028386 , @ronlieb wrote: > i backed up to your reverted patch, and applied this one. > I see some new errors > > libomptarget :: amdgcn-amd-amdhsa :: mapping/declare_mapper_target.cpp > libomptarget :: amdgcn

[PATCH] D110670: [Sema] Allow comparisons between different ms ptr size address space types.

akhuang created this revision. akhuang added a reviewer: aaron.ballman. akhuang requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. We're currently using address spaces to implement __ptr32/__ptr64 attributes; this patch fixes a bug where clang

[PATCH] D110668: [clang-cl] Accept `#pragma warning(disable : N)` for some N

thakis updated this revision to Diff 375749. thakis added a comment. some clang-format CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110668/new/ https://reviews.llvm.org/D110668 Files: clang/include/clang/Basic/CLWarnings.h clang/include/clang/Driver/Options.td clang/lib/Basic/CL

[PATCH] D110668: [clang-cl] Accept `#pragma warning(disable : N)` for some N

thakis updated this revision to Diff 375750. thakis added a comment. tweak CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110668/new/ https://reviews.llvm.org/D110668 Files: clang/include/clang/Basic/CLWarnings.h clang/include/clang/Driver/Options.td clang/lib/Basic/CLWarnings.cpp

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path and include/x86_64-linux-gnu/c++/v1 libc++ path

MaskRay updated this revision to Diff 375751. MaskRay marked an inline comment as done. MaskRay retitled this revision from "[Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path" to "[Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtim

[clang] 25c3032 - [AIX] Change the linkage of profiling counter/data to be private

Author: Jinsong Ji Date: 2021-09-29T00:47:25Z New Revision: 25c30324e953c1c0011d1ab7529991db5c6ba741 URL: https://github.com/llvm/llvm-project/commit/25c30324e953c1c0011d1ab7529991db5c6ba741 DIFF: https://github.com/llvm/llvm-project/commit/25c30324e953c1c0011d1ab7529991db5c6ba741.diff LOG: [A

[PATCH] D110422: [AIX] Change the linkage of profiling counter/data to be private

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG25c30324e953: [AIX] Change the linkage of profiling counter/data to be private (authored by jsji). Repository: rG LLVM Github Monorepo CHANGES SI

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path and include/x86_64-linux-gnu/c++/v1 libc++ path

phosek added a comment. The reason I removed this behavior in D101194 , aside from extra overhead introduced by the extra checks, is that we've seen cases where people would have both paths on their system which lead to difficult to diagnose issues where Clang

[PATCH] D110663: [Driver] Support Debian multiarch style lib/clang/14.0.0/x86_64-linux-gnu runtime path and include/x86_64-linux-gnu/c++/v1 libc++ path

MaskRay updated this revision to Diff 375753. MaskRay added a comment. fix getRuntimePath test libclang_rt.builtins.a Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110663/new/ https://reviews.llvm.org/D110663 Files: clang/lib/Driver/ToolChain.cp

[PATCH] D110671: [AIX] Enable PGO without LTO

jsji created this revision. jsji added reviewers: PowerPC, Whitney. Herald added subscribers: wenlei, inglorion. jsji requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. On AIX, we relied on LTO to merge the csects for profiling data/counter se

<    1   2   3   >