[PATCH] D96049: [Timer] On macOS count number of executed instructions

2021-02-11 Thread Alex Hoppen via Phabricator via cfe-commits
ahoppen added inline comments. Comment at: llvm/CMakeLists.txt:656 + list(APPEND CMAKE_REQUIRED_LIBRARIES proc) +endif() + thakis wrote: > also, any reason to not do this in `llvm/cmake/config-ix.cmake` where all the > other checks like this are? No, I just did

[PATCH] D94621: [clang-tidy] add concurrency-async-fs

2021-02-11 Thread Aaron Ballman via Phabricator via cfe-commits
aaron.ballman added a comment. In D94621#2554228 , @segoon wrote: > alexfh, aaron.ballman, hi! Any comments on the patch? Have you run this check over large real-world code bases beyond Yandex.Taxi? I think this is likely to be a maintenance burden becau

[PATCH] D96507: [clangd] Move command handlers into a map in ClangdLSPServer. NFC

2021-02-11 Thread Kadir Cetinkaya via Phabricator via cfe-commits
kadircet accepted this revision. kadircet added a comment. This revision is now accepted and ready to land. thanks, lgtm! Comment at: clang-tools-extra/clangd/ClangdLSPServer.cpp:587 + std::vector Commands; + llvm::copy(CommandHandlers.keys(), std::back_inserter(Commands)); +

[PATCH] D96363: Mark output as text if it is really text

2021-02-11 Thread Jonas Devlieghere via Phabricator via cfe-commits
JDevlieghere accepted this revision. JDevlieghere added a comment. This revision is now accepted and ready to land. LGTM with two inline nits. Comment at: clang/lib/Frontend/Rewrite/FrontendActions.cpp:188 std::unique_ptr OS = - CI.createDefaultOutputFile(true, getCurre

[PATCH] D94779: [Clang] Ensure vector predication loop metadata is always emitted when pragma is specified.

2021-02-11 Thread Dave Green via Phabricator via cfe-commits
dmgreen added a comment. Thanks for your comments. This patch is not intended to have no consequence - it's just not being communicated well in the commit message. The problem at the moment, with mainline clang, is that specifying: #pragma clang loop vectorize_width(4) vectorize_predicate(ena

[PATCH] D95772: [DebugInfo] Normalize paths by removing unnecessary dots

2021-02-11 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. In D95772#2534103 , @dblaikie wrote: > Does this address @rnk's feedback about symlinks? ( > https://reviews.llvm.org/D87657#2296028 ) It doesn't, I think if we want to handle that case, we'd need to use `sys::fs::real_path` rath

[PATCH] D96487: Restore diagnostic handler after CodeGenAction::ExecuteAction

2021-02-11 Thread Rong Xu via Phabricator via cfe-commits
xur accepted this revision. xur added a comment. This revision is now accepted and ready to land. This looks fine to me. Thanks for fixing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96487/new/ https://reviews.llvm.org/D96487 __

[clang] a680bc3 - [clang][Arm] Fix handling of -Wa,-implicit-it=

2021-02-11 Thread Nick Desaulniers via cfe-commits
Author: Nick Desaulniers Date: 2021-02-11T10:51:25-08:00 New Revision: a680bc3a31d36d321ccf3801bdcff74d58842bfa URL: https://github.com/llvm/llvm-project/commit/a680bc3a31d36d321ccf3801bdcff74d58842bfa DIFF: https://github.com/llvm/llvm-project/commit/a680bc3a31d36d321ccf3801bdcff74d58842bfa.di

[PATCH] D96285: [clang][Arm] Fix handling of -Wa,-implicit-it=

2021-02-11 Thread Nick Desaulniers 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 rGa680bc3a31d3: [clang][Arm] Fix handling of -Wa,-implicit-it= (authored by nickdesaulniers). Repository: rG LLVM Github Monorepo CHANGES SINCE LAS

[PATCH] D89274: [WebAssembly] Use the new crt1-command.o if present.

2021-02-11 Thread Sam Clegg via Phabricator via cfe-commits
sbc100 accepted this revision. sbc100 added a comment. This revision is now accepted and ready to land. Ok so we can see this as an interm thing. I think I'm OK with that. Could you add a comment about that, or at least say why we want to support both the old crt1 and the new crt1-command at t

[PATCH] D95928: [OpenMP] Delay more diagnostics of potentially non-emitted code

2021-02-11 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95928/new/ https://reviews.llvm.org/D95928 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi

[PATCH] D96427: Support multi-configuration generators correctly in several config files

2021-02-11 Thread Raphael Isemann via Phabricator via cfe-commits
teemperor added a comment. In D96427#2555251 , @JDevlieghere wrote: > LGTM. I'm surprised the `dsymutil` one slipped through the cracks, we have a > bot that should (?) be testing this configuration: > http://green.lab.llvm.org/green/view/LLDB/job/lldb-

[PATCH] D92808: [ObjC][ARC] Use operand bundle 'clang.arc.rv' instead of explicitly emitting retainRV or claimRV calls in the IR

2021-02-11 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith added a comment. In D92808#2555868 , @ahatanak wrote: > For example, if SCCP just does a normal RAUW on the following call, which is > taken from the example I posted, > > %r = call i8* @foo() [ "clang.arc.rv"(i64 1, i8* %r) ] > > will become

[PATCH] D96248: [OpenMP][AMDGPU] Add support for linking libomptarget bitcode

2021-02-11 Thread Shilei Tian via Phabricator via cfe-commits
tianshilei1992 accepted this revision. tianshilei1992 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/D96248/new/ https://reviews.llvm.org/D96248 _

[PATCH] D96495: [clangd] Retire the cross-file-rename command-line flag.

2021-02-11 Thread Haojian Wu via Phabricator via cfe-commits
hokein added a comment. In D96495#2556897 , @njames93 wrote: > Those test failures look related. oh, right. the failure tests rely on the default cross-file flag, fixed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.

[PATCH] D96495: [clangd] Retire the cross-file-rename command-line flag.

2021-02-11 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 323084. hokein added a comment. fix failure tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96495/new/ https://reviews.llvm.org/D96495 Files: clang-tools-extra/clangd/refactor/Rename.h clang-tools-extr

[PATCH] D94779: [Clang] Ensure vector predication loop metadata is always emitted when pragma is specified.

2021-02-11 Thread Michael Kruse via Phabricator via cfe-commits
Meinersbur accepted this revision. Meinersbur added a comment. Thank you for the clarification. It would be great if that could be communicated in the summary/commit message. It's sounds like emitting `llvm.loop.vectorize.predicate.enable` when vectorization is disabled is the motivation behind

[PATCH] D92808: [ObjC][ARC] Use operand bundle 'clang.arc.rv' instead of explicitly emitting retainRV or claimRV calls in the IR

2021-02-11 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith added a comment. In D92808#2556542 , @fhahn wrote: > @dexonsmith @rjmccall What do you think? Would you be happy with iterating > on the suggestions in tree? I don't feel strongly either way; happy for it to land as-is (bugs fixed) and itera

[PATCH] D96363: Mark output as text if it is really text

2021-02-11 Thread Abhina Sree via Phabricator via cfe-commits
abhina.sreeskantharajan updated this revision to Diff 323093. abhina.sreeskantharajan added a comment. Thanks for reviewing! Addressing inline comments by adding /*Binary*/ comment. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96363/new/ https://r

[PATCH] D96363: Mark output as text if it is really text

2021-02-11 Thread Jonas Devlieghere via Phabricator via cfe-commits
JDevlieghere accepted this revision. JDevlieghere added a comment. Thanks. LGTM! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96363/new/ https://reviews.llvm.org/D96363 ___ cfe-commits mailing list cfe-

[PATCH] D93023: Replace deprecated %T in 2 tests.

2021-02-11 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land. LGTM. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93023/new/ https://reviews.llvm.org/D93023 ___ cfe-commits mailing list cfe-commits@

[clang-tools-extra] 573348a - [clangd] Retire the cross-file-rename command-line flag.

2021-02-11 Thread Haojian Wu via cfe-commits
Author: Haojian Wu Date: 2021-02-11T20:28:50+01:00 New Revision: 573348ab9b281221e0e78376c233d1898ed0bf68 URL: https://github.com/llvm/llvm-project/commit/573348ab9b281221e0e78376c233d1898ed0bf68 DIFF: https://github.com/llvm/llvm-project/commit/573348ab9b281221e0e78376c233d1898ed0bf68.diff LO

[PATCH] D96495: [clangd] Retire the cross-file-rename command-line flag.

2021-02-11 Thread Haojian Wu 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 rG573348ab9b28: [clangd] Retire the cross-file-rename command-line flag. (authored by hokein). Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D92808: [ObjC][ARC] Use operand bundle 'clang.arc.rv' instead of explicitly emitting retainRV or claimRV calls in the IR

2021-02-11 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak marked 3 inline comments as done. ahatanak added a comment. In D92808#2557634 , @dexonsmith wrote: > In D92808#2555868 , @ahatanak wrote: > >> For example, if SCCP just does a normal RAUW on the following c

[PATCH] D95753: Store compilation dir separately in coverage mapping

2021-02-11 Thread Petr Hosek via Phabricator via cfe-commits
phosek updated this revision to Diff 323106. phosek marked 2 inline comments as done. phosek added a comment. Herald added a project: Sanitizers. Herald added a subscriber: Sanitizers. I have addressed all the comments and fixed all tests. I'm happy to slice this into multiple patches if you pref

[PATCH] D92808: [ObjC][ARC] Use operand bundle 'clang.arc.rv' instead of explicitly emitting retainRV or claimRV calls in the IR

2021-02-11 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak updated this revision to Diff 323107. ahatanak added a comment. Address all review comments except for the ones about the bundle name. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92808/new/ https://reviews.llvm.org/D92808 Files: clang

[PATCH] D92808: [ObjC][ARC] Use operand bundle 'clang.arc.rv' instead of explicitly emitting retainRV or claimRV calls in the IR

2021-02-11 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak added a comment. In D92808#2557132 , @fhahn wrote: > Another thing I noticed that there's verifier support missing. I think we > should at least check that only a single `clang.arc.rv` bundle is specified > (https://github.com/llvm/llvm-project/

[PATCH] D92808: [ObjC][ARC] Use operand bundle 'clang.arc.rv' instead of explicitly emitting retainRV or claimRV calls in the IR

2021-02-11 Thread Florian Hahn via Phabricator via cfe-commits
fhahn accepted this revision. fhahn added a comment. This revision is now accepted and ready to land. LGTM as initial step, but it would be good to adjust the name as Duncan suggested before landing. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D92

[PATCH] D96538: [SYCL] Ignore file-scope asm during device-side SYCL compilation.

2021-02-11 Thread Artur Gainullin via Phabricator via cfe-commits
ArturGainullin created this revision. ArturGainullin added reviewers: bader, elizabethandrews. Herald added subscribers: Anastasia, ebevhan, yaxunl. ArturGainullin requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github

[clang] 7b48326 - NFCI. With the move to the new pass manager by default, sanitize-coverage.c is now passing on ARM.

2021-02-11 Thread Douglas Yung via cfe-commits
Author: Douglas Yung Date: 2021-02-11T13:18:18-08:00 New Revision: 7b4832648a6339c798f1f72bbc88b1ee41e9a338 URL: https://github.com/llvm/llvm-project/commit/7b4832648a6339c798f1f72bbc88b1ee41e9a338 DIFF: https://github.com/llvm/llvm-project/commit/7b4832648a6339c798f1f72bbc88b1ee41e9a338.diff

[PATCH] D96542: [clang-tidy] Fix `TransformerClangTidyCheck`'s handling of include insertions.

2021-02-11 Thread Yitzhak Mandelbaum via Phabricator via cfe-commits
ymandel created this revision. ymandel added a reviewer: alexfh. Herald added a subscriber: xazax.hun. ymandel requested review of this revision. Herald added a project: clang. Currently, all include insertions are directed to the main file. However, Transformer rules can specify alternative desti

[PATCH] D96406: [Msan, NewPM] Reduce size of msan binaries

2021-02-11 Thread Evgenii Stepanov via Phabricator via cfe-commits
eugenis accepted this revision. eugenis added a comment. This revision is now accepted and ready to land. LGTM, please add a TODO for more passes Comment at: clang/lib/CodeGen/BackendUtil.cpp:1291 + // general purpose optimization passes. + FPM.addPass

[PATCH] D95753: Store compilation dir separately in coverage mapping

2021-02-11 Thread Vedant Kumar via Phabricator via cfe-commits
vsk added a comment. Two minor missing items: - Please include a binary test that verifies llvm-cov can prepare a report for a binary containing a Version6 coverage blob (for compatibility testing). - Please also include a short blurb in docs/CoverageMappingFormat.rst explaining the format chan

[PATCH] D96544: [clangd] Extract binding of typed->untyped LSP handlers to LSPBinder. NFC

2021-02-11 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: kadircet. Herald added subscribers: usaxena95, arphaman, mgorny. sammccall requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang. The goal is to allow the LSP bind

[PATCH] D68590: [clangd] Improve hover scopes for Objective-C code

2021-02-11 Thread David Goldman via Phabricator via cfe-commits
dgoldman updated this revision to Diff 323141. dgoldman marked 7 inline comments as done. dgoldman added a comment. - Address review comments Will move over to AST.cpp in next update Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68590/new/ https:/

[PATCH] D68590: [clangd] Improve hover scopes for Objective-C code

2021-02-11 Thread David Goldman via Phabricator via cfe-commits
dgoldman added inline comments. Comment at: clang-tools-extra/clangd/Hover.cpp:64 +static llvm::StringRef getNameForObjCInterface(const ObjCInterfaceDecl *ID) { + return ID ? ID->getName() : "<>"; sammccall wrote: > this function's name doesn't really describe

[PATCH] D68590: [clangd] Improve hover scopes for Objective-C code

2021-02-11 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. Comment at: clang-tools-extra/clangd/Hover.cpp:64 +static llvm::StringRef getNameForObjCInterface(const ObjCInterfaceDecl *ID) { + return ID ? ID->getName() : "<>"; dgoldman wrote: > sammccall

[PATCH] D68590: [clangd] Improve hover scopes for Objective-C code

2021-02-11 Thread David Goldman via Phabricator via cfe-commits
dgoldman updated this revision to Diff 323151. dgoldman added a comment. - Move over to AST.cpp Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68590/new/ https://reviews.llvm.org/D68590 Files: clang-tools-extra/clangd/AST.cpp clang-tools-extra/

[clang] 61cca0f - [AArch64] Adding Neon Sm3 & Sm4 Intrinsics

2021-02-11 Thread Pengxuan Zheng via cfe-commits
Author: Pengxuan Zheng Date: 2021-02-11T14:20:20-08:00 New Revision: 61cca0f2e5bbb6045bb27b822e34cd39c9c1acb1 URL: https://github.com/llvm/llvm-project/commit/61cca0f2e5bbb6045bb27b822e34cd39c9c1acb1 DIFF: https://github.com/llvm/llvm-project/commit/61cca0f2e5bbb6045bb27b822e34cd39c9c1acb1.diff

[PATCH] D95655: [AArch64] Adding Neon Sm3 & Sm4 Intrinsics

2021-02-11 Thread Pengxuan Zheng via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG61cca0f2e5bb: [AArch64] Adding Neon Sm3 & Sm4 Intrinsics (authored by pzheng). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95655/new/ https://reviews.llvm

[PATCH] D93023: Replace deprecated %T in 2 tests.

2021-02-11 Thread Hafiz Abid Qadeer via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG60bed4ab57d5: Replace deprecated %T in 2 tests. (authored by abidh). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93023/new/ https://reviews.llvm.org/D9302

[clang] 60bed4a - Replace deprecated %T in 2 tests.

2021-02-11 Thread Hafiz Abid Qadeer via cfe-commits
Author: Hafiz Abid Qadeer Date: 2021-02-11T22:21:21Z New Revision: 60bed4ab57d562d5770cc9c24a8fcb243208f5e5 URL: https://github.com/llvm/llvm-project/commit/60bed4ab57d562d5770cc9c24a8fcb243208f5e5 DIFF: https://github.com/llvm/llvm-project/commit/60bed4ab57d562d5770cc9c24a8fcb243208f5e5.diff

[PATCH] D68590: [clangd] Improve hover scopes for Objective-C code

2021-02-11 Thread David Goldman via Phabricator via cfe-commits
dgoldman updated this revision to Diff 323156. dgoldman marked an inline comment as done. dgoldman added a comment. - Support protocols Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68590/new/ https://reviews.llvm.org/D68590 Files: clang-tools-e

[PATCH] D96344: [flang][driver] Add options for -fdefault* and -flarge-sizes

2021-02-11 Thread Arnamoy B via Phabricator via cfe-commits
arnamoy10 added inline comments. Comment at: flang/lib/Frontend/CompilerInvocation.cpp:261 + diags.getCustomDiagID(clang::DiagnosticsEngine::Error, + "Use of `-fdefault-double-8` requires `-fdefault-real-8`"); + diags.Report(diagID); --

[PATCH] D96244: [clangd] Introduce Modules

2021-02-11 Thread Sam McCall via Phabricator via cfe-commits
sammccall added a comment. I'm a little bit nervous about adding this with *no* usage, but it keeps the patch size down :-) Comment at: clang-tools-extra/clangd/Module.cpp:1 +//===--- Module.cpp - Main clangd server code *- C++-*-===// +//

[clang] f9c05fc - [WebAssembly] Use the new crt1-command.o if present.

2021-02-11 Thread Dan Gohman via cfe-commits
Author: Dan Gohman Date: 2021-02-11T14:44:37-08:00 New Revision: f9c05fc391458e455e8e4d4108a7f270f2f7bedc URL: https://github.com/llvm/llvm-project/commit/f9c05fc391458e455e8e4d4108a7f270f2f7bedc DIFF: https://github.com/llvm/llvm-project/commit/f9c05fc391458e455e8e4d4108a7f270f2f7bedc.diff LO

[PATCH] D89274: [WebAssembly] Use the new crt1-command.o if present.

2021-02-11 Thread Dan Gohman 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 rGf9c05fc39145: [WebAssembly] Use the new crt1-command.o if present. (authored by sunfish). Changed prior to commit: https://reviews.llvm.org/D89274

[PATCH] D95396: Improve static_assert/_Static_assert diagnostics

2021-02-11 Thread Richard Smith - zygoloid via Phabricator via cfe-commits
rsmith added a comment. FYI, https://reviews.llvm.org/D17444 captures some of the history here. Comment at: clang/lib/Parse/ParseDeclCXX.cpp:874-876 +if (!getLangOpts().CPlusPlus) + Diag(Tok, diag::warn_cxx_static_assert_in_c) + << FixItHint::CreateReplacement

[PATCH] D96481: [NFC] Extract function which registers sanitizer passes

2021-02-11 Thread Vitaly Buka 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 rGf2f59d2a0607: [NFC] Extract function which registers sanitizer passes (authored by vitalybuka). Repository: rG LLVM Github Monorepo CHANGES SINCE

[clang] f2f59d2 - [NFC] Extract function which registers sanitizer passes

2021-02-11 Thread Vitaly Buka via cfe-commits
Author: Vitaly Buka Date: 2021-02-11T15:29:48-08:00 New Revision: f2f59d2a060788f17040ad924ee2d11da0e215c8 URL: https://github.com/llvm/llvm-project/commit/f2f59d2a060788f17040ad924ee2d11da0e215c8 DIFF: https://github.com/llvm/llvm-project/commit/f2f59d2a060788f17040ad924ee2d11da0e215c8.diff L

[PATCH] D86546: [compiler-rt][builtins] Use explicitly-sized integer types for LibCalls

2021-02-11 Thread Ayke via Phabricator via cfe-commits
aykevl added a comment. This looks good to me, although I would like someone else to take a look as well. I can confirm that these changes fix an issue on AVR: this patch (together with D86547 ) make `__floatsisf` correct on AVR while it would previously do some

[PATCH] D96406: [Msan, NewPM] Reduce size of msan binaries

2021-02-11 Thread Vitaly Buka 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 rG686b65f85f22: [Msan, NewPM] Reduce size of msan binaries (authored by vitalybuka). Changed prior to commit: https://reviews.llvm.org/D96406?vs=322

[clang] 686b65f - [Msan, NewPM] Reduce size of msan binaries

2021-02-11 Thread Vitaly Buka via cfe-commits
Author: Vitaly Buka Date: 2021-02-11T16:07:18-08:00 New Revision: 686b65f85f22c70e082753cd05e8572d22e3991c URL: https://github.com/llvm/llvm-project/commit/686b65f85f22c70e082753cd05e8572d22e3991c DIFF: https://github.com/llvm/llvm-project/commit/686b65f85f22c70e082753cd05e8572d22e3991c.diff L

[PATCH] D86547: [compiler-rt][builtins] Use c[tl]zsi macro instead of __builtin_c[tl]z

2021-02-11 Thread Ayke via Phabricator via cfe-commits
aykevl added a comment. I checked this, and it seems good to me although I would like someone else to also take a look. In D86547#2287353 , @atrosinenko wrote: > In D86547#2284095 , @MaskRay wrote: > >> The `(aWid

[PATCH] D94640: adds more checks to -Wfree-nonheap-object

2021-02-11 Thread Christopher Di Bella via Phabricator via cfe-commits
cjdb added inline comments. Comment at: clang/test/Analysis/free.c:84 + // expected-warning@-1{{Argument to free() is a block, which is not memory allocated by malloc()}} + // expected-warning@-2{{attempt to call free on non-heap object : block expression}} } ---

[PATCH] D96456: [ThinLTO, NewPM] Add Config::OptPassBuilderHook

2021-02-11 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka updated this revision to Diff 323202. vitalybuka marked 5 inline comments as done. vitalybuka added a comment. update Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96456/new/ https://reviews.llvm.org/D96456 Files: clang/lib/CodeGen/Ba

[PATCH] D96456: [ThinLTO, NewPM] Register sanitizers with OptimizerLastPassBuilderHook

2021-02-11 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:1328 +CodeGenOpts.ThinLTOIndexFile.empty()) { + // This is testing distributed ThinLTO PostLink. O0 called optimized in + // PreLink. tejohnson wrote: > I don't unde

[PATCH] D96456: [ThinLTO, NewPM] Register sanitizers with OptimizerLastPassBuilderHook

2021-02-11 Thread Vitaly Buka via Phabricator via cfe-commits
vitalybuka added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:1328 +CodeGenOpts.ThinLTOIndexFile.empty()) { + // This is testing distributed ThinLTO PostLink. O0 called optimized in + // PreLink. vitalybuka wrote: > tejohnson w

[PATCH] D96456: [ThinLTO, NewPM] Register sanitizers with OptimizerLastPassBuilderHook

2021-02-11 Thread Arthur Eubanks via Phabricator via cfe-commits
aeubanks added inline comments. Comment at: clang/lib/CodeGen/BackendUtil.cpp:1070-1071 +// ThinLTOIndexFile is provideds so we must be in ThinLTO PostLink. +// For -O0 ThinLTO PreLink does basic optimization and triggers +// OptimizerLastEPCallbacks. Post

[clang] e3cd3a3 - Partially Revert "scan-view: Remove Reporter.py and associated AppleScript files"

2021-02-11 Thread Tom Stellard via cfe-commits
Author: Tom Stellard Date: 2021-02-11T19:10:46-08:00 New Revision: e3cd3a3c91524c957e06bb0170343548f02b6842 URL: https://github.com/llvm/llvm-project/commit/e3cd3a3c91524c957e06bb0170343548f02b6842 DIFF: https://github.com/llvm/llvm-project/commit/e3cd3a3c91524c957e06bb0170343548f02b6842.diff

[PATCH] D96367: Partially Revert "scan-view: Remove Reporter.py and associated AppleScript files"

2021-02-11 Thread Tom Stellard 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 rGe3cd3a3c9152: Partially Revert "scan-view: Remove Reporter.py and associated AppleScript… (authored by tstellar). Changed prior to commit: https:/

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

2021-02-11 Thread Dan Liew via Phabricator via cfe-commits
delcypher created this revision. delcypher added reviewers: arphaman, kubamracek, yln, aralisza, kcc, vitalybuka. Herald added subscribers: dexonsmith, dang. Herald added a reviewer: jansvoboda11. delcypher requested review of this revision. Herald added a project: clang. Herald added a subscriber:

[PATCH] D96572: [Clang][ASan] Introduce `-fsanitize-address-destructor-kind=` driver & frontend option.

2021-02-11 Thread Dan Liew via Phabricator via cfe-commits
delcypher updated this revision to Diff 323213. delcypher added a comment. clang-format fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96572/new/ https://reviews.llvm.org/D96572 Files: clang/include/clang/Basic/CodeGenOptions.def clang/incl

[PATCH] D96573: [Clang][ASan] Teach Clang to not emit ASan module destructors when compiling with `-mkernel` or `-fapple-kext`.

2021-02-11 Thread Dan Liew via Phabricator via cfe-commits
delcypher created this revision. delcypher added reviewers: arphaman, kubamracek, yln, aralisza, kcc, vitalybuka. delcypher requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. rdar://71609176 Repository: rG LLVM Github Monorepo https://rev

[PATCH] D92808: [ObjC][ARC] Use operand bundle 'clang.arc.attachedcall' instead of explicitly emitting retainRV or claimRV calls in the IR

2021-02-11 Thread Akira Hatanaka via Phabricator via cfe-commits
ahatanak updated this revision to Diff 323218. ahatanak retitled this revision from "[ObjC][ARC] Use operand bundle 'clang.arc.rv' instead of explicitly emitting retainRV or claimRV calls in the IR" to "[ObjC][ARC] Use operand bundle 'clang.arc.attachedcall' instead of explicitly emitting retain

[clang] 79401b4 - [OpenMP][AMDGPU] Add support for linking libomptarget bitcode

2021-02-11 Thread Pushpinder Singh via cfe-commits
Author: Pushpinder Singh Date: 2021-02-12T00:42:41-05:00 New Revision: 79401b43ce4e3aa856c50e78b38327e3ff4ae9eb URL: https://github.com/llvm/llvm-project/commit/79401b43ce4e3aa856c50e78b38327e3ff4ae9eb DIFF: https://github.com/llvm/llvm-project/commit/79401b43ce4e3aa856c50e78b38327e3ff4ae9eb.di

[PATCH] D96248: [OpenMP][AMDGPU] Add support for linking libomptarget bitcode

2021-02-11 Thread Pushpinder Singh 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 rG79401b43ce4e: [OpenMP][AMDGPU] Add support for linking libomptarget bitcode (authored by Pushpinder Singh , committe

[PATCH] D96538: [SYCL] Ignore file-scope asm during device-side SYCL compilation.

2021-02-11 Thread Alexey Bader via Phabricator via cfe-commits
bader accepted this revision. bader added a comment. This revision is now accepted and ready to land. LGTM. Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96538/new/ https://reviews.llvm.org/D96538 __

[PATCH] D93003: [libunwind] unw_* alias fixes for ELF and Mach-O

2021-02-11 Thread Ryan Prichard via Phabricator via cfe-commits
rprichard updated this revision to Diff 323221. rprichard edited the summary of this revision. rprichard added a comment. Update libunwind/src/BUILD.gn for macro name change. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93003/new/ https://reviews.

[PATCH] D93003: [libunwind] unw_* alias fixes for ELF and Mach-O

2021-02-11 Thread Ryan Prichard via Phabricator via cfe-commits
rprichard updated this revision to Diff 323222. rprichard added a comment. Rebase and fix merge conflict in gn file. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93003/new/ https://reviews.llvm.org/D93003 Files: clang/cmake/caches/Fuchsia-stage

<    1   2