[clang] 87e68ca - Improve relnotes for the DWARFv5 default change

Author: David Blaikie Date: 2022-01-25T09:56:07-08:00 New Revision: 87e68cae50d7ca28975ca6fb456cf0ab2ac915a1 URL: https://github.com/llvm/llvm-project/commit/87e68cae50d7ca28975ca6fb456cf0ab2ac915a1 DIFF: https://github.com/llvm/llvm-project/commit/87e68cae50d7ca28975ca6fb456cf0ab2ac915a1.diff

[PATCH] D118152: [clang][deps] Adapt test to be compatible when the assembler is called by default

Jake-Egan added inline comments. Comment at: clang/test/ClangScanDeps/headerwithdirname.cpp:16 -// CHECK: headerwithdirname_input.o +// CHECK: headerwithdirname_input{{.*}}{{.o|.s}} // CHECK-NEXT: headerwithdirname_input.cpp jansvoboda11 wrote: > Why is `{{.*}

[PATCH] D118109: [clang] Replace `std::vector` use in Syntax

dexonsmith accepted this revision. dexonsmith added a comment. LGTM too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118109/new/ https://reviews.llvm.org/D118109 ___ cfe-commits mailing list cfe-commit

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (PR53360)

aeubanks added a comment. The existing implementation only inlines top level call sites, which doesn't match gcc where all calls are recursively inlined [1]. gcc's implementation makes more sense IMO, only inlining top level call sites doesn't seem super useful. I'd vote for properly implement

[PATCH] D117603: [clang] Don't typo-fix an expression in a SFINAE context

Quuxplusone added reviewers: ChuanqiXu, urnathan, hokein, dblaikie. Quuxplusone added a comment. Add some more reviewers who've touched SemaCXX recently. Ping! If no objections are forthcoming, I'd like to land this on Friday morning. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST AC

[clang-tools-extra] 19d7a0b - [clang-tidy] [bugprone-assert-side-effect] Ignore list for functions/methods

Author: Zinovy Nis Date: 2022-01-25T21:04:07+03:00 New Revision: 19d7a0b47b6849923576845f87a3278e012e049b URL: https://github.com/llvm/llvm-project/commit/19d7a0b47b6849923576845f87a3278e012e049b DIFF: https://github.com/llvm/llvm-project/commit/19d7a0b47b6849923576845f87a3278e012e049b.diff LO

[PATCH] D116478: [clang-tidy] A semicolon-separated list of the names of functions or methods to be considered as not having side-effects

This revision was automatically updated to reflect the committed changes. Closed by commit rG19d7a0b47b68: [clang-tidy] [bugprone-assert-side-effect] Ignore list for functions/methods (authored by zinovy.nis). Changed prior to commit: https://reviews.llvm.org/D116478?vs=402199&id=402956#toc Re

[PATCH] D116478: [clang-tidy] A semicolon-separated list of the names of functions or methods to be considered as not having side-effects

zinovy.nis marked an inline comment as done. zinovy.nis added inline comments. Comment at: clang-tools-extra/docs/ReleaseNotes.rst:161 + check now supports an ``IgnoredFunctions`` option to explicitly consider + the specified semicolon-separated functions list as not any having

[PATCH] D117603: [clang] Don't typo-fix an expression in a SFINAE context

urnathan accepted this revision. urnathan added a comment. This revision is now accepted and ready to land. go for it Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117603/new/ https://reviews.llvm.org/D117603 __

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (PR53360)

xbolva00 added a comment. There is also second use case which I mentioned and I am interested in. >> I'd vote for properly implementing [1] or removing our support for flatten. I would rather prefer to restore atleast something which was working before new PM and then improve it, instead of fu

[PATCH] D115031: [AST] Print NTTP args as string-literals when possible

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

[PATCH] D117137: [Driver] Add CUDA support for --offload param

tra added a comment. In D117137#3268548 , @linjamaki wrote: > SPIR-V target requires that the OS and the environment type is unknown (see > TargetInfo::AllocateTarget and BaseSPIRTargetInfo). The problem is that LLVM's triple parser will set `UnknownVen

[PATCH] D114095: [clang][lex] Include tracking: simplify and move to preprocessor

jansvoboda11 updated this revision to Diff 402968. jansvoboda11 added a comment. 1. Move `HeaderSearch::getFileInfo()` call into `Preprocessor::markIncluded()`. 2. Switch from bitvector to vector of file IDs in AST files. 3. Remove unnecessary usages of `auto`. Repository: rG LLVM Github Monor

[PATCH] D114095: [clang][lex] Include tracking: simplify and move to preprocessor

jansvoboda11 added a comment. In D114095#3263086 , @vsapsai wrote: > My understanding was that we were going with ID vectors approach and not with > bitvectors, and the current code still uses bitvectors. Also I have a minor > readability concern about

[PATCH] D56303: [clang-tidy] Recognize labelled statements when simplifying boolean exprs

LegalizeAdulthood added a comment. In D56303#3266252 , @njames93 wrote: > A large portion of the changes, especially in the checks implementation file, > appear to be NFC stylistic or formatting only fixes. While these changes are > generally good, they

[PATCH] D118153: [CUDA][HIP] Do not treat host var address as constant in device compilation

tra added a comment. LGTM. Do we need to do anything special about `__managed__` vars? Comment at: clang/lib/AST/ExprConstant.cpp:2224 + Info.getCtx().getLangOpts().CUDAIsDevice) { +if (!Var->hasAttr() && +!Var->hasAttr() && Nit: N

[PATCH] D117238: [C2x] Add BITINT_MAXWIDTH support

aaron.ballman updated this revision to Diff 402971. aaron.ballman added a comment. Herald added subscribers: pcwang-thead, luke957, s.egerton, simoncook, fedor.sergeev. Updated to allow targets to specify the max bitwidth. I was aware that the x86 backend had issues with doing division on larger

[PATCH] D117616: GCC ABI Compatibility: Preserve alignment of non-pod members in packed structs

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

[PATCH] D115867: [C++20] [Coroutines] Warning for always_inline coroutine

Quuxplusone accepted this revision. Quuxplusone added a comment. This revision is now accepted and ready to land. LGTM now, modulo my suggested edits (and the necessary corresponding edits in the test case). I don't think I'm really qualified to accept, but as nobody else is looking, and my name

[clang] c39d22d - [CMake] Set sanitizer test C++ library on Linux

Author: Petr Hosek Date: 2022-01-25T11:09:48-08:00 New Revision: c39d22d1968cf07e54b5816ba76dccef8acaace1 URL: https://github.com/llvm/llvm-project/commit/c39d22d1968cf07e54b5816ba76dccef8acaace1 DIFF: https://github.com/llvm/llvm-project/commit/c39d22d1968cf07e54b5816ba76dccef8acaace1.diff LO

[PATCH] D118161: [CMake] Set sanitizer test C++ library on Linux

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGc39d22d1968c: [CMake] Set sanitizer test C++ library on Linux (authored by phosek). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D118095: [AVR][clang] Reject non assembly source files for the avr1 family

MaskRay added a comment. I don't know the AVR specific thing but having this toolchain-specific diagnostic in the toolchain-specific file seems the right direction. Comment at: clang/lib/Driver/ToolChains/AVR.cpp:418 +// a '.s' suffix. +for (unsigned I = 0; I < DriverA

[PATCH] D118169: [NFC][CodeGen] Use llvm::DenseMap for DeferredDecls

yurai007 created this revision. yurai007 added reviewers: nikic, xbolva00, erichkeane, craig.topper, aeubanks, Tyker. Herald added a subscriber: pengfei. yurai007 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. CodeGenModule::DeferredDecls

[PATCH] D117616: GCC ABI Compatibility: Preserve alignment of non-pod members in packed structs

tstellar added a comment. So to summarize: this will change the ABI of clang built objects from clang13 -> clang14 in order to make clang14 compatible with gcc? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117616/new/ https://reviews.llvm.org/D11

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (PR53360)

aeubanks added a comment. sounds good for the purposes of going back to what we had before, but we should reconsider the attribute support Comment at: llvm/lib/Transforms/IPO/AlwaysInliner.cpp:95 - // Remember to try and delete this function afterward. This both avoids

[PATCH] D117616: GCC ABI Compatibility: Preserve alignment of non-pod members in packed structs

dblaikie added a comment. In D117616#3270205 , @tstellar wrote: > So to summarize: this will change the ABI of clang built objects from clang13 > -> clang14 in order to make clang14 compatible with gcc? That's the idea, yes. (only for non-pod members of

[clang] 0ad19a8 - [CUDA, NVPTX] Corrected fragment size for tf32 LD B matrix.

Author: JackAKirk Date: 2022-01-25T11:29:19-08:00 New Revision: 0ad19a833177861be55fefaff725ab89c8695d01 URL: https://github.com/llvm/llvm-project/commit/0ad19a833177861be55fefaff725ab89c8695d01 DIFF: https://github.com/llvm/llvm-project/commit/0ad19a833177861be55fefaff725ab89c8695d01.diff LOG

[PATCH] D118023: Corrected fragment size for tf32 LD B matrix.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0ad19a833177: [CUDA,NVPTX] Corrected fragment size for tf32 LD B matrix. (authored by JackAKirk, committed by tra). Repository: rG LLVM Github Mon

[PATCH] D116377: [libTooling] Adds more support for constructing object access expressions.

ymandel updated this revision to Diff 402984. ymandel added a comment. Herald added a subscriber: JDevlieghere. fix bad previous snapshot Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116377/new/ https://reviews.llvm.org/D116377 Files: clang/inc

[clang] 0944c19 - [libTooling] Adds more support for constructing object access expressions.

Author: Yitzhak Mandelbaum Date: 2022-01-25T19:43:36Z New Revision: 0944c196c58f62299540983e2d10cd7bef60691a URL: https://github.com/llvm/llvm-project/commit/0944c196c58f62299540983e2d10cd7bef60691a DIFF: https://github.com/llvm/llvm-project/commit/0944c196c58f62299540983e2d10cd7bef60691a.diff

[PATCH] D116377: [libTooling] Adds more support for constructing object access expressions.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0944c196c58f: [libTooling] Adds more support for constructing object access expressions. (authored by ymandel). Repository: rG LLVM Github Monorep

[PATCH] D114439: [Annotation] Allow parameter pack expansions in annotate attribute

aaron.ballman added inline comments. Comment at: clang/lib/Parse/ParseDecl.cpp:497 +// Pack expansion is only allowed in the variadic argument at the end. +const size_t attrNumArgs = attributeNumberOfArguments(*AttrName); +if (ArgExprs.size() + I + 1 < att

[PATCH] D118153: [CUDA][HIP] Do not treat host var address as constant in device compilation

yaxunl added a comment. In D118153#3270122 , @tra wrote: > LGTM. > > Do we need to do anything special about `__managed__` vars? Right `__managed__` var is special. Its address is set by runtime, therefore it is not a constant. nvcc does not treat it as

[clang] 491c154 - [analyzer] Don't specify PLUGIN_TOOL for analyzer plugins

Author: Aleksandr Platonov Date: 2022-01-25T23:05:00+03:00 New Revision: 491c154677bcf0fba3c91fdaa7897a48ab605327 URL: https://github.com/llvm/llvm-project/commit/491c154677bcf0fba3c91fdaa7897a48ab605327 DIFF: https://github.com/llvm/llvm-project/commit/491c154677bcf0fba3c91fdaa7897a48ab605327.

[PATCH] D116966: [analyzer] Don't specify PLUGIN_TOOL for analyzer plugins

This revision was automatically updated to reflect the committed changes. Closed by commit rG491c154677bc: [analyzer] Don't specify PLUGIN_TOOL for analyzer plugins (authored by ArcsinX). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116966/new/ htt

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

xbolva00 updated this revision to Diff 402994. xbolva00 retitled this revision from "[AlwaysInliner] Enable call site inlining to make flatten attribute working again (PR53360)" to "[AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)". Herald added a subscri

[PATCH] D45438: [CodeView] Enable debugging of captured variables within C++ lambdas

rnk added inline comments. Comment at: lib/CodeGen/CGDebugInfo.cpp:812-814 + // CodeView types with C++ mangling need a type identifier. + if (CGM.getCodeGenOpts().EmitCodeView) +return true; dblaikie wrote: > bwyma wrote: > > dblaikie wrote: > > > rnk wrot

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

xbolva00 added a comment. Thanks, addressed review comments. > we should reconsider the attribute support I agree. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117965/new/ https://reviews.llvm.org/D117965 ___ cfe-commits mailing list cfe-c

[PATCH] D117238: [C2x] Add BITINT_MAXWIDTH support

erichkeane added inline comments. Comment at: clang/lib/Frontend/InitPreprocessor.cpp:917 + Builder.defineMacro("__BITINT_MAXWIDTH__", Twine(TI.getMaxBitIntWidth())); + Could you do an assert that TI.getMaxBitIntWidth() is not greater than the LLVM maximum (j

[PATCH] D117939: [clang-tidy] Add more documentation about check development

aaron.ballman added a comment. Aside from the unit testing bit, I think this is fantastic! (And the unit testing bit may also be fantastic, I just think it needs more explicit discussion with a wider audience.) Comment at: clang-tools-extra/docs/clang-tidy/Contributing.rst:30

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

xbolva00 updated this revision to Diff 402997. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117965/new/ https://reviews.llvm.org/D117965 Files: clang/test/CodeGen/flatten.c clang/test/CodeGenCXX/flatten.cpp llvm/lib/Transforms/IPO/AlwaysInliner.cpp llvm/test/Transforms/Coroutine

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

xbolva00 added inline comments. Comment at: llvm/lib/Transforms/IPO/AlwaysInliner.cpp:95 - // Remember to try and delete this function afterward. This both avoids - // re-walking the rest of the module and avoids dealing with any iterator - // invalidation issues

[PATCH] D45438: [CodeView] Enable debugging of captured variables within C++ lambdas

dblaikie added inline comments. Comment at: lib/CodeGen/CGDebugInfo.cpp:812-814 + // CodeView types with C++ mangling need a type identifier. + if (CGM.getCodeGenOpts().EmitCodeView) +return true; rnk wrote: > dblaikie wrote: > > bwyma wrote: > > > dblaikie

[PATCH] D114095: [clang][lex] Include tracking: simplify and move to preprocessor

vsapsai accepted this revision. vsapsai added a comment. This revision is now accepted and ready to land. Looks good to me. Thanks for working on it! Though I haven't built it locally and it can be useful for pre-merge checks to finish. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D45438: [CodeView] Enable debugging of captured variables within C++ lambdas

rnk added inline comments. Comment at: lib/CodeGen/CGDebugInfo.cpp:812-814 + // CodeView types with C++ mangling need a type identifier. + if (CGM.getCodeGenOpts().EmitCodeView) +return true; dblaikie wrote: > rnk wrote: > > dblaikie wrote: > > > bwyma wrot

[PATCH] D118178: [clang][dataflow] Allow clients to disable built-in transfer functions.

ymandel created this revision. ymandel added a reviewer: sgatev. Herald added subscribers: tschuett, steakhal. ymandel requested review of this revision. Herald added a project: clang. These built-in functions build the (sophisticated) model of the code's memory. This model isn't used by all analy

[PATCH] D117238: [C2x] Add BITINT_MAXWIDTH support

aaron.ballman updated this revision to Diff 403008. aaron.ballman added a comment. Updated based on review feedback. I added an additional assert to test that the value picked by the target meets the requirements from the C standard. (We already had a test case for it in lit, but no reason not t

[PATCH] D112915: [clang][modules] Track included files per submodule

vsapsai added a comment. Can you please rebase this change after D114095 lands? Overall looks good but I want to take one final look and triggering the pre-merge checks will be useful. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:/

[PATCH] D117616: GCC ABI Compatibility: Preserve alignment of non-pod members in packed structs

rnk added a comment. In addition to the ABI compatibility concern, IMO, GCC's behavior here is better and more user friendly: - C++ classes have methods - calling a method on a field takes the address of a field - if the field is in a packed struct, it may be misaligned - misaligned loads and st

[PATCH] D117238: [C2x] Add BITINT_MAXWIDTH support

erichkeane accepted this revision. erichkeane added a comment. Accept again, thanks! Note the purpose of the assert is to make sure that a backend doesn't make a bad decision, and the test for 'max size' is only validating 1 backend (AND has now been changed to >128 I think). CHANGES SINCE LA

[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 added a comment. Ping @phosek Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110663/new/ https://reviews.llvm.org/D110663 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

aeubanks accepted this revision. aeubanks added inline comments. This revision is now accepted and ready to land. Comment at: llvm/test/Transforms/Coroutines/coro-retcon-once-private.ll:6 -; CHECK: define internal { i8*, i32 } @f(i8* %buffer, i32* %array) -; CHECK-NEXT: entry:

[PATCH] D117616: GCC ABI Compatibility: Preserve alignment of non-pod members in packed structs

dblaikie added a subscriber: probinson. dblaikie added a comment. @probinson for Sony ABI input Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117616/new/ https://reviews.llvm.org/D117616 ___ cfe-commits

[PATCH] D117616: GCC ABI Compatibility: Preserve alignment of non-pod members in packed structs

tstellar added a comment. Can you add a release note to go along with this change? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117616/new/ https://reviews.llvm.org/D117616 ___ cfe-commits mailing list

[PATCH] D118169: [NFC][CodeGen] Use llvm::DenseMap for DeferredDecls

aeubanks accepted this revision. aeubanks added a comment. This revision is now accepted and ready to land. https://llvm-compile-time-tracker.com/compare.php?from=c39d22d1968cf07e54b5816ba76dccef8acaace1&to=1958f5a5fb5be3fb5b6bad5079a2f3485db0b0e9&stat=instructions mostly neutral but a tiny bit of

[PATCH] D118169: [NFC][CodeGen] Use llvm::DenseMap for DeferredDecls

erichkeane accepted this revision. erichkeane added a comment. This looks fine to change, and seems like a slight compile timeimprovement. It seems that the only thing we use this for is .find, .erase, and operator[], all of which don't depend on iterator order, so this should not cause any pro

[PATCH] D118150: [clang] Fix serialized diagnostics edge-cases

Bigcheese accepted this revision. Bigcheese 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/D118150/new/ https://reviews.llvm.org/D118150 _

[PATCH] D118184: [ASan] Not lininking asan_static library for DSO.

kstoimenov created this revision. kstoimenov requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Whithout this change DSOs fail to link because of missing asan_report_(load|store)n functions. Repository: rG LLVM Github Monorepo https://re

[PATCH] D117939: [clang-tidy] Add more documentation about check development

ymandel added a comment. Overall, this looks fantastic! You may want to consider (in a separate patch) mentioning godbolt.org, which is a great UI for interacting with clang-query and the AST. Example configuration: https://godbolt.org/z/v88W8ET19 In D117939#3266234

[clang] ceec438 - [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

Author: Dávid Bolvanský Date: 2022-01-25T22:55:30+01:00 New Revision: ceec4383681c42bfd3d06a6913ce7554bea160b0 URL: https://github.com/llvm/llvm-project/commit/ceec4383681c42bfd3d06a6913ce7554bea160b0 DIFF: https://github.com/llvm/llvm-project/commit/ceec4383681c42bfd3d06a6913ce7554bea160b0.dif

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGceec4383681c: [AlwaysInliner] Enable call site inlining to make flatten attribute working… (authored by xbolva00). Changed prior to commit: https:

[PATCH] D117939: [clang-tidy] Add more documentation about check development

LegalizeAdulthood marked an inline comment as done. LegalizeAdulthood added inline comments. Comment at: clang-tools-extra/docs/clang-tidy/Contributing.rst:362 +Private custom matchers are a good example of auxiliary support code for your check +that is best tested with a unit t

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

thakis added a comment. Could this have caused http://45.33.8.238/linux/66347/step_7.txt ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117965/new/ https://reviews.llvm.org/D117965 ___ cfe-commits maili

[PATCH] D117965: [AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)

xbolva00 added a comment. In D117965#3270744 , @thakis wrote: > Could this have caused http://45.33.8.238/linux/66347/step_7.txt ? Yeah, possible as inliner removes internal unused functions (small change before commit, missed check-clang). I will rev

[clang] 90f185c - Revert "[AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)"

Author: Dávid Bolvanský Date: 2022-01-25T23:13:46+01:00 New Revision: 90f185c964d05e9f3081bb4acc734b3932db221a URL: https://github.com/llvm/llvm-project/commit/90f185c964d05e9f3081bb4acc734b3932db221a DIFF: https://github.com/llvm/llvm-project/commit/90f185c964d05e9f3081bb4acc734b3932db221a.dif

[clang] 8ba9c79 - Add support for sycl_special_class attribute.

Author: Zahira Ammarguellat Date: 2022-01-25T14:17:09-08:00 New Revision: 8ba9c794feb30cd969b9776c39873def10c51bff URL: https://github.com/llvm/llvm-project/commit/8ba9c794feb30cd969b9776c39873def10c51bff DIFF: https://github.com/llvm/llvm-project/commit/8ba9c794feb30cd969b9776c39873def10c51bff

[PATCH] D117830: [HeaderSearch] Track framework name in LookupFile

jansvoboda11 added a comment. I don't have any suggestions here except for running the patch through `clang-format`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117830/new/ https://reviews.llvm.org/D117830 __

[clang] 493509a - [NFC] DeclCXX: Fix -Wreorder-ctor

Author: Jordan Rupprecht Date: 2022-01-25T14:29:35-08:00 New Revision: 493509a40ad1653feb779861badd3ee59fc4a79c URL: https://github.com/llvm/llvm-project/commit/493509a40ad1653feb779861badd3ee59fc4a79c DIFF: https://github.com/llvm/llvm-project/commit/493509a40ad1653feb779861badd3ee59fc4a79c.di

[PATCH] D118196: [syntax][pseudo] Implement LR parsing table.

hokein created this revision. hokein added a reviewer: sammccall. Herald added a subscriber: mgorny. hokein requested review of this revision. Herald added a project: clang. This patch introduces a dense implementation of the LR parsing table, which is used by LR parsers. We implement a SLR(1) par

[PATCH] D118197: [OpenMP] Replace sysmtem call to `llc` with target machine

jhuber6 created this revision. jhuber6 added reviewers: jdoerfert, JonChesterfield, ronlieb, saiislam. Herald added subscribers: mikhail.ramalho, guansong, yaxunl, mgorny. jhuber6 requested review of this revision. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. Thi

[PATCH] D118198: [OpenMP] Remove call to 'clang-offload-wrapper' binary

jhuber6 created this revision. jhuber6 added reviewers: jdoerfert, JonChesterfield, ronlieb, saiislam. Herald added subscribers: guansong, yaxunl, mgorny. jhuber6 requested review of this revision. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: clang. This patch removes t

[PATCH] D118199: [AArch64] ACLE feature macro for Armv8.8-A MOPS

tyb0807 created this revision. Herald added a subscriber: kristof.beyls. tyb0807 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This introduces the new __ARM_FEATURE_MOPS ACLE feature test macro, which signals the availability of the new A

[PATCH] D116875: [clang-tidy] Add performance-inefficient-array-traversal check

njames93 updated this revision to Diff 403050. njames93 added a comment. Simplify check logic to only detect loop inits with 0 and incriment with 1. Functionality can be introduced with later patches. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D11

[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 subscriber: hvdijk. phosek added a comment. In D110663#3270509 , @MaskRay wrote: > Ping @phosek I apologize about the belated response, but I wanted to test this change first given the potentially large blast radius. To clarify, when I t

[PATCH] D118084: [CUDA, NVPTX] Pass byval aggregates directly

jdoerfert added a comment. The RFC discussion concluded this should be fine wrt. the interoperability use cases we want to support. Code change looks good but I have one question. Comment at: clang/lib/CodeGen/TargetInfo.cpp:7183 } -return getNaturalAlignIndirect(Ty,

[clang-tools-extra] ce94432 - [clangd] Add designator inlay hints for initializer lists.

Author: Sam McCall Date: 2022-01-26T00:35:29+01:00 New Revision: ce94432702bf42a0b95a2693aa47177f37dd0bb3 URL: https://github.com/llvm/llvm-project/commit/ce94432702bf42a0b95a2693aa47177f37dd0bb3 DIFF: https://github.com/llvm/llvm-project/commit/ce94432702bf42a0b95a2693aa47177f37dd0bb3.diff LO

[PATCH] D116786: [clangd] Add designator inlay hints for initializer lists.

This revision was automatically updated to reflect the committed changes. Closed by commit rGce94432702bf: [clangd] Add designator inlay hints for initializer lists. (authored by sammccall). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116786/new/

[PATCH] D118184: [ASan] Not linking asan_static library for DSO.

kda accepted this revision. kda added a comment. This revision is now accepted and ready to land. unit test? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118184/new/ https://reviews.llvm.org/D118184 ___

[clang] fe30370 - Reland "[AlwaysInliner] Enable call site inlining to make flatten attribute working again (#53360)"

Author: Dávid Bolvanský Date: 2022-01-26T01:11:06+01:00 New Revision: fe30370b007e7efa1bd1f39a3189b27ae4e5fcbe URL: https://github.com/llvm/llvm-project/commit/fe30370b007e7efa1bd1f39a3189b27ae4e5fcbe DIFF: https://github.com/llvm/llvm-project/commit/fe30370b007e7efa1bd1f39a3189b27ae4e5fcbe.dif

[clang] cf730d8 - [ASan] Not linking asan_static library for DSO.

Author: Kirill Stoimenov Date: 2022-01-26T00:13:26Z New Revision: cf730d8ce1341ba593144df2e2bc8411238e04c3 URL: https://github.com/llvm/llvm-project/commit/cf730d8ce1341ba593144df2e2bc8411238e04c3 DIFF: https://github.com/llvm/llvm-project/commit/cf730d8ce1341ba593144df2e2bc8411238e04c3.diff L

[PATCH] D118184: [ASan] Not linking asan_static library for DSO.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGcf730d8ce134: [ASan] Not linking asan_static library for DSO. (authored by kstoimenov). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST AC

[PATCH] D117262: [NFC] Store Address's alignment into PointerIntPairs

aeubanks updated this revision to Diff 403070. aeubanks added a comment. only use PointerIntPair when alignof(Value*) >= 8 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117262/new/ https://reviews.llvm.org/D117262 Files: clang/lib/CodeGen/Addres

[PATCH] D118084: [CUDA, NVPTX] Pass byval aggregates directly

tra planned changes to this revision. tra added a comment. Getting rid of byval helps getting rid of locals in quite a few places, but runs into a new problem. 😕 Looks like this change does have unexpected side-effects. When we need to dynamically index into a struct passed directly, there's no

[PATCH] D118084: [CUDA, NVPTX] Pass byval aggregates directly

jdoerfert added a subscriber: lebedev.ri. jdoerfert added a comment. @lebedev.ri wanted to teach SROA how to deal with dynamic indices before, IIRC. It seems to be generally useful. This patch can wait till then? Comment at: clang/lib/CodeGen/TargetInfo.cpp:7193 EIT-

[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 added a comment. No worries! Thanks for the reply. I use Debian and want it to work well and support the `-DLLVM_ENABLE_PER_TARGET_RUNTIME_DIR=on` direction. Currently, `-DLLVM_DEFAULT_TARGET_TRIPLE=aarch64-linux-gnu` (Debian style multiarch triple, so no `-unknown` or `-pc`) does not w

[PATCH] D118084: [CUDA, NVPTX] Pass byval aggregates directly

tra added a comment. In D118084#3271073 , @jdoerfert wrote: > @lebedev.ri wanted to teach SROA how to deal with dynamic indices before, > IIRC. It seems to be generally useful. Interesting. I'd like to hear more. > This patch can wait till then? Yes.

[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 added inline comments. Comment at: clang/lib/Driver/Driver.cpp:428 +const Driver &D, StringRef TargetTriple, const ArgList &Args, +llvm::Triple *UnnormalizedTriple = nullptr, StringRef DarwinArchName = "") { // FIXME: Already done in Compilation *Driver::Build

[PATCH] D118205: [ASan] Added a unit test for D118184.

kstoimenov created this revision. kstoimenov added a reviewer: kda. kstoimenov requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. To make sure the the libraries are there for executable and not there for DSOs. Repository: rG LLVM Github Mo

[PATCH] D118205: [ASan] Added a unit test for D118184.

kstoimenov updated this revision to Diff 403083. kstoimenov added a comment. Fixed order. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118205/new/ https://reviews.llvm.org/D118205 Files: clang/test/Driver/sanitizer-ld.c Index: clang/test/Driv

[PATCH] D118205: [ASan] Added a unit test for D118184.

kda accepted this revision. kda added a comment. This revision is now accepted and ready to land. thank you. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118205/new/ https://reviews.llvm.org/D118205 ___

[clang] 69da422 - [ASan] Added a unit test for D118184.

Author: Kirill Stoimenov Date: 2022-01-26T01:20:02Z New Revision: 69da422bdaa684fbd14c95b06f40ac57910642d9 URL: https://github.com/llvm/llvm-project/commit/69da422bdaa684fbd14c95b06f40ac57910642d9 DIFF: https://github.com/llvm/llvm-project/commit/69da422bdaa684fbd14c95b06f40ac57910642d9.diff L

[PATCH] D118205: [ASan] Added a unit test for D118184.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG69da422bdaa6: [ASan] Added a unit test for D118184. (authored by kstoimenov). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION htt

[PATCH] D117647: [RISCV] Add passthru operand for RVV nomask load intrinsics.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG9273378b8576: [RISCV] Add the passthru operand for RVV nomask load intrinsics. (authored by khchen). Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D112493: [OpenMP][NFCI] Record the location string as part of the ident_t

jdoerfert abandoned this revision. jdoerfert added a comment. Committed partially as 944aa0421cb7da3aa764b2a108ea25ef8bceb979 , rest is NFC. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.l

[PATCH] D117295: [clang][sema] Allow unnamed decls in C++20 module export{} blocks

ChuanqiXu added a comment. Oh, it looks like we couldn't go on since the part of PR1766R1 is not adopted: https://github.com/cplusplus/draft/issues/5237 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117295/new/ https://reviews.llvm.org/D117295 _

[PATCH] D117972: [PowerPC] Fix SSE translation on FreeBSD

qiucf added a comment. Thanks for the fix! Comment at: clang/lib/Headers/ppc_wrappers/mm_malloc.h:22 #else +#if defined(__linux__) extern "C" int posix_memalign (void **, size_t, size_t) throw (); Actually I just realised we don't need this guard, Clang alrea

[PATCH] D118199: [AArch64] ACLE feature macro for Armv8.8-A MOPS

nickdesaulniers added a comment. > This patch supersedes the one from https://reviews.llvm.org/D116160. Can that one be abandoned then? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118199/new/ https://reviews.llvm.org/D118199 ___

[PATCH] D117972: [PowerPC] Fix SSE translation on FreeBSD

pkubaj updated this revision to Diff 403103. pkubaj added a comment. Address review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D117972/new/ https://reviews.llvm.org/D117972 Files: clang/lib/Headers/ppc_wrappers/mm_malloc.h Index: clang/lib/

[PATCH] D117616: GCC ABI Compatibility: Preserve alignment of non-pod members in packed structs

rjmccall added a comment. > In addition to the ABI compatibility concern, IMO, GCC's behavior here is > better and more user friendly: > > - C++ classes have methods > - calling a method on a field takes the address of a field I mean, this argument is hardly limited to non-POD types. I suppose

[clang] 5c1f7b2 - [C++20] [Modules] Only check decls under namespace scope in CheckRedeclarationExported

Author: Chuanqi Xu Date: 2022-01-26T10:54:52+08:00 New Revision: 5c1f7b296ac0dddeca02891976e6ab5cfc006719 URL: https://github.com/llvm/llvm-project/commit/5c1f7b296ac0dddeca02891976e6ab5cfc006719 DIFF: https://github.com/llvm/llvm-project/commit/5c1f7b296ac0dddeca02891976e6ab5cfc006719.diff LO

<    1   2   3   >