[PATCH] D97915: [Coroutines] Handle overaligned frame allocation

ychen updated this revision to Diff 341418. ychen added a comment. - Handle deallocation. - Fix tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97915/new/ https://reviews.llvm.org/D97915 Files: clang/lib/CodeGen/CGBuiltin.cpp clang/lib/Co

[PATCH] D97915: [Coroutines] Handle overaligned frame allocation

ychen added a comment. @rjmccall the patch is on the large side. I'll submit a separate patch for the Sema part about searching for two allocators. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97915/new/ https://reviews.llvm.org/D97915

[PATCH] D100739: [Coroutines] Handle overaligned frame allocation (2)

ychen abandoned this revision. ychen added a comment. Reopened D97915 to address the feedbacks. Close this one. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100739/new/ https://reviews.llvm.org/D100739 _

[clang] 6a0283d - [NewPM] Add an option to dump pass structure

Author: Evgeny Leviant Date: 2021-04-29T10:29:42+03:00 New Revision: 6a0283d0d23cc8b056005caa31097dfb78853548 URL: https://github.com/llvm/llvm-project/commit/6a0283d0d23cc8b056005caa31097dfb78853548 DIFF: https://github.com/llvm/llvm-project/commit/6a0283d0d23cc8b056005caa31097dfb78853548.diff

[PATCH] D99599: [NewPM] Add an option to dump pass structure

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6a0283d0d23c: [NewPM] Add an option to dump pass structure (authored by evgeny777). Herald added a project: clang. Herald added a subscriber: cfe-com

[PATCH] D100667: [clang] Fix assert() crash when checking undeduced arg alignment

hokein accepted this revision. hokein added a comment. still LG Comment at: clang/test/SemaCXX/recovery-expr-type.cpp:29 // verify that "field has incomplete type" diagnostic is suppressed. - typeof(foo(42)) var; // expected-error {{no matching function}} + typeof(foo(42))

[clang] 6e5082b - [clang-format] Fix build on gcc < 7 introduced in rG9363aa9.

Author: Marek Kurdej Date: 2021-04-29T09:57:34+02:00 New Revision: 6e5082bbc498ab7d68178ea883203b38f6cd47fb URL: https://github.com/llvm/llvm-project/commit/6e5082bbc498ab7d68178ea883203b38f6cd47fb DIFF: https://github.com/llvm/llvm-project/commit/6e5082bbc498ab7d68178ea883203b38f6cd47fb.diff

[clang] 40c2d61 - [clang-format] Fix build on gcc < 7 introduced in rG9363aa9.

Author: Marek Kurdej Date: 2021-04-29T10:07:04+02:00 New Revision: 40c2d6188b08f7a1bdd23a8cfdfea8fa998bdff6 URL: https://github.com/llvm/llvm-project/commit/40c2d6188b08f7a1bdd23a8cfdfea8fa998bdff6 DIFF: https://github.com/llvm/llvm-project/commit/40c2d6188b08f7a1bdd23a8cfdfea8fa998bdff6.diff

[PATCH] D91054: [Clang][OpenMP] Frontend work for sections - D89671

AMDChirag updated this revision to Diff 341436. AMDChirag added a comment. Rebased to HEAD Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91054/new/ https://reviews.llvm.org/D91054 Files: clang/lib/CodeGen/CGOpenMPRuntime.cpp clang/lib/CodeGen/

[PATCH] D95976: [OpenMP] Simplify offloading parallel call codegen

protze.joachim added a comment. Please update the test with a NFC commit. Comment at: openmp/libomptarget/test/offloading/bug49779.cpp:1-5 +// RUN: %libomptarget-compilexx-run-and-check-aarch64-unknown-linux-gnu +// RUN: %libomptarget-compilexx-run-and-check-powerpc64-ibm-linux

[PATCH] D101515: clang-format: [JS] handle "off" in imports

mprobst created this revision. mprobst added reviewers: krasimir, h-joo. mprobst requested review of this revision. Herald added a project: clang. Previously, the JavaScript import sorter would ignore `// clang-format off` and `on` comments. This change fixes that. It tracks whether formatting is

[PATCH] D101516: Introduce clangd-server-monitor tool

kbobyrev created this revision. kbobyrev added a reviewer: kadircet. Herald added subscribers: usaxena95, arphaman, mgorny. kbobyrev requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra. Repository: rG LLVM Gi

[PATCH] D101209: [PowerPC] Provide fastmath sqrt and div functions in altivec.h

nemanjai updated this revision to Diff 341456. nemanjai added a comment. Changed `rsqrt` to be an actual reciprocal rather than just a refined `sqrt` estimate. I have verified that the code generated is equivalent to gcc's and the results produced are the same. Repository: rG LLVM Github Mo

[clang] 0ff41c2 - Update libstdc++ hack comment

Author: Nathan Sidwell Date: 2021-04-29T03:57:10-07:00 New Revision: 0ff41c2ebc9904f881c958f9006bbf2b6bdc5d1e URL: https://github.com/llvm/llvm-project/commit/0ff41c2ebc9904f881c958f9006bbf2b6bdc5d1e DIFF: https://github.com/llvm/llvm-project/commit/0ff41c2ebc9904f881c958f9006bbf2b6bdc5d1e.diff

[PATCH] D101441: Update libstdc++ hack comment

This revision was automatically updated to reflect the committed changes. Closed by commit rG0ff41c2ebc99: Update libstdc++ hack comment (authored by urnathan). Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D101519: [C++4OpenCL] Fix reinterpret_cast of vectors and types with address spaces

olestrohm created this revision. olestrohm added reviewers: Anastasia, svenvh. olestrohm added a project: clang. Herald added subscribers: ldrumm, yaxunl. olestrohm requested review of this revision. Herald added a subscriber: cfe-commits. This fixes two issues with reinterpret_cast in C++ for Ope

[PATCH] D100872: Use OpenFlags instead of boolean to set a file as text/binary

abhina.sreeskantharajan added inline comments. Comment at: clang/include/clang/Frontend/CompilerInstance.h:738-740 + createOutputFileImpl(StringRef OutputPath, llvm::sys::fs::OpenFlags Flags, bool RemoveFileOnSignal, bool UseTemporary,

[PATCH] D101462: Make it possible for targets to define their own MCObjectFileInfo

flip1995 created this revision. flip1995 added reviewers: MaskRay, rnk, asb. Herald added subscribers: frasercrmck, dexonsmith, luismarques, apazos, sameer.abuasal, s.egerton, Jim, jocewei, rupprecht, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, atanasyan, edward-jones, zzheng, jrtc27, gbedw

[PATCH] D101462: Make it possible for targets to define their own MCObjectFileInfo

jrtc27 added a comment. Herald added a subscriber: JDevlieghere. What's the benefit of less-aligned functions? Is that not more likely to get poor performance due to cache line straddling of the first instruction? Regardless, the functional change should be separated out from the refactoring.

[PATCH] D99484: Use `GNUInstallDirs` to support custom installation dirs.

Ericson2314 updated this revision to Diff 341213. Ericson2314 added a comment. Fix stray ':' typo Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99484/new/ https://reviews.llvm.org/D99484 Files: clang-tools-extra/clang-doc/tool/CMakeLists.txt c

[PATCH] D89013: [libcxx] Support per-target __config_site in per-target runtime build

phosek updated this revision to Diff 341262. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89013/new/ https://reviews.llvm.org/D89013 Files: clang/lib/Driver/ToolChains/Fuchsia.cpp clang/lib/Driver/ToolChains/Gnu.cpp clang/test/Driver/Inputs/basic_fuchsia_tree/include/aarch64-unkn

[PATCH] D89013: [libcxx] Support per-target __config_site in per-target runtime build

MaskRay added inline comments. Comment at: clang/test/Driver/linux-header-search.cpp:16 // CHECK-BASIC-LIBCXX-SYSROOT: "-isysroot" "[[SYSROOT:[^"]+]]" +// CHECK-BASIC-LIBCXX-SYSROOT: "-internal-isystem" "[[SYSROOT]]/usr/include/x86_64-unknown-linux-gnu/c++/v1" // CHECK-BASIC-L

[PATCH] D89013: [libcxx] Support per-target __config_site in per-target runtime build

MaskRay accepted this revision. MaskRay added a comment. The path component `x86_64-unknown-linux-gnu` (not multiarch `x86_64-linux-gnu`) looks good to me CHANGES SINCE LAST ACTION https://reviews.llvm.org/D89013/new/ https://reviews.llvm.org/D89013 _

[PATCH] D89013: [libcxx] Support per-target __config_site in per-target runtime build

This revision was not accepted when it landed; it landed in state "Needs Review". This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGea12d779bc23: [libc++] Support per-target __config_site in per-tar

[PATCH] D99484: Use `GNUInstallDirs` to support custom installation dirs.

Ericson2314 updated this revision to Diff 341326. Ericson2314 added a comment. Fix bug makeing polly path full, and rebase fixing conflicts Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99484/new/ https://reviews.llvm.org/D99484 Files: clang-too

[PATCH] D101462: Make it possible for targets to define their own MCObjectFileInfo

MaskRay added a comment. The refactoring part seems useful on its own. The controversial 4-byte alignment part should be split. Doesn't GNU as use 4 for most architectures as well? Why do you want to change it? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews

[PATCH] D99484: Use `GNUInstallDirs` to support custom installation dirs.

Ericson2314 updated this revision to Diff 341339. Ericson2314 added a comment. Resubmit now prior diff is rebased, so CI doesn't trip up Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99484/new/ https://reviews.llvm.org/D99484 Files: clang-tools-

[PATCH] D99484: Use `GNUInstallDirs` to support custom installation dirs.

Meinersbur added a comment. In D99484#2724055 , @Ericson2314 wrote: > Fix bug makeing polly path full, and rebase fixing conflicts What was the problem? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99484/n

[PATCH] D99484: Use `GNUInstallDirs` to support custom installation dirs.

Ericson2314 added a comment. In D99484#2724343 , @Meinersbur wrote: > In D99484#2724055 , @Ericson2314 > wrote: > >> Fix bug makeing polly path full, and rebase fixing conflicts > > What was the problem? I effecti

[PATCH] D99484: Use `GNUInstallDirs` to support custom installation dirs.

Meinersbur added a comment. In D99484#2724435 , @Ericson2314 wrote: > I effectively replaced `${CMAKE_INSTALL_PREFIX}/lib` with > `${CMAKE_INSTALL_LIBDIR}`, which evaluates (by default) merely to `lib`. The > fix was `${CMAKE_INSTALL_FULL_LIBDIR}` which

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

jansvoboda11 accepted this revision. jansvoboda11 added a comment. In D101491#2724025 , @delcypher wrote: > @jansvoboda11 Should I use the the alias feature so that the old > `-fsanitize-address-destructor-kind` argument is still parsed? I'm not sure >

[PATCH] D101526: [analyzer][StdLibraryFunctionsChecker] Add NoteTags for applied arg constraints

martong created this revision. martong added reviewers: vsavchenko, NoQ, steakhal. Herald added subscribers: ASDenysPetrov, gamesh411, dkrupp, donat.nagy, Szelethus, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, xazax.hun, whisperity. Herald added a reviewer: Szelethus. marton

[PATCH] D101462: Make it possible for targets to define their own MCObjectFileInfo

flip1995 added a comment. Thanks for the review! I already thought that I will have to split this up, so I made the commits self contained so I'll do that. One question before I start: Where should I split this? Should I only split out the RISC-V patch and leave the changes that targets can def

[PATCH] D99488: [SYCL][Doc] Add design document for SYCL mode

Anastasia added a comment. If I build docs now I get the following output: llvm-project/build-doc/tools/clang/docs/SYCLSupport.rst:102: WARNING: Error in "code-block" directive: 1 argument(s) required, 0 supplied. .. code-block:: TODO: add support for `__attribute__((opencl_glob

[clang] 8fb0d6d - [OpenCL][Docs] Describe extension for legacy atomics with generic addr space.

Author: Anastasia Stulova Date: 2021-04-29T14:02:34+01:00 New Revision: 8fb0d6df11e4d2b4a534c96fcc1b55459940151e URL: https://github.com/llvm/llvm-project/commit/8fb0d6df11e4d2b4a534c96fcc1b55459940151e DIFF: https://github.com/llvm/llvm-project/commit/8fb0d6df11e4d2b4a534c96fcc1b55459940151e.d

[PATCH] D101089: [OpenCL] Document legacy atomics with generic address space

This revision was automatically updated to reflect the committed changes. Closed by commit rG8fb0d6df11e4: [OpenCL][Docs] Describe extension for legacy atomics with generic addr space. (authored by Anastasia). Herald added a subscriber: ldrumm. Herald added a project: clang. Changed prior to comm

[clang] 1ed6e87 - [OpenCL][Docs] Misc updates to C++ for OpenCL and offline compilation

Author: Anastasia Stulova Date: 2021-04-29T14:15:07+01:00 New Revision: 1ed6e87ab02db4fb9379b80c820bebad9869aa4d URL: https://github.com/llvm/llvm-project/commit/1ed6e87ab02db4fb9379b80c820bebad9869aa4d DIFF: https://github.com/llvm/llvm-project/commit/1ed6e87ab02db4fb9379b80c820bebad9869aa4d.d

[PATCH] D101092: [OpenCL][Docs] Misc updates about C++ for OpenCL and offline compilation

This revision was automatically updated to reflect the committed changes. Closed by commit rG1ed6e87ab02d: [OpenCL][Docs] Misc updates to C++ for OpenCL and offline compilation (authored by Anastasia). Herald added a subscriber: ldrumm. Herald added a project: clang. Repository: rG LLVM Github

[PATCH] D99488: [SYCL][Doc] Add design document for SYCL mode

bader added a comment. In D99488#2725435 , @Anastasia wrote: > If I build docs now I get the following output: > > llvm-project/build-doc/tools/clang/docs/SYCLSupport.rst:102: WARNING: Error > in "code-block" directive: > 1 argument(s) required, 0 sup

[PATCH] D91054: [Clang][OpenMP] Frontend work for sections - D89671

AMDChirag updated this revision to Diff 341503. AMDChirag added a comment. Rebase to retrigger build The build failed previously - seemingly with a test case that has nothing to do with this patch. Retriggering and Rebasing the commit to ensure the same. Repository: rG LLVM Github Monorepo

[PATCH] D100092: [clang-tidy] cppcoreguidelines-declare-loop-variable-in-the-initializer: a new check

DNS320 updated this revision to Diff 341492. DNS320 marked 7 inline comments as done. DNS320 added a comment. I updated the check according to the last review. The check should now fully implement the defined enforcement chapter from the ES.74 C++ Core Guideline. About searching for declRefExpr

[PATCH] D100092: [clang-tidy] cppcoreguidelines-declare-loop-variable-in-the-initializer: a new check

DNS320 added inline comments. Comment at: clang-tools-extra/clang-tidy/cppcoreguidelines/DeclareLoopVariableInTheInitializerCheck.cpp:28 + + diag(MatchedForStmt->getBeginLoc(), + "Prefer to declare a loop variable in the initializer part of a " Eugene.Zel

[PATCH] D101168: [C++4OpenCL] Add clang extension for non-portable kernel parameters

Anastasia added inline comments. Comment at: clang/docs/LanguageExtensions.rst:1808 +With this extension it is possible to enable the use of some restricted types in kernel parameters. +The restrictions can be relaxed using regular OpenCL extension pragma mechanism +detailed in

[PATCH] D100092: [clang-tidy] cppcoreguidelines-declare-loop-variable-in-the-initializer: a new check

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-tidy/cppcoreguidelines/DeclareLoopVariableInTheInitializerCheck.cpp:37 + bool VisitDeclRefExpr(DeclRefExpr *D) { +if (const VarDecl *To = dyn_cast(D->getDecl())) { + if (To == MatchedDecl &&

[clang] 3eb2be6 - Unbreak no-asserts testing

Author: David Zarzycki Date: 2021-04-29T10:01:37-04:00 New Revision: 3eb2be67b997ea62f47dbe90a62e828ecfb266a8 URL: https://github.com/llvm/llvm-project/commit/3eb2be67b997ea62f47dbe90a62e828ecfb266a8 DIFF: https://github.com/llvm/llvm-project/commit/3eb2be67b997ea62f47dbe90a62e828ecfb266a8.diff

[clang] c204106 - [Clang][OpenMP] Frontend work for sections - D89671

Author: Chirag Khandelwal Date: 2021-04-29T19:52:27+05:30 New Revision: c20410618827b7870fbc86d45ff8e9b11bc169b4 URL: https://github.com/llvm/llvm-project/commit/c20410618827b7870fbc86d45ff8e9b11bc169b4 DIFF: https://github.com/llvm/llvm-project/commit/c20410618827b7870fbc86d45ff8e9b11bc169b4.d

[PATCH] D91054: [Clang][OpenMP] Frontend work for sections - D89671

This revision was automatically updated to reflect the committed changes. Closed by commit rGc20410618827: [Clang][OpenMP] Frontend work for sections - D89671 (authored by AMDChirag). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91054/new/ https://

[PATCH] D101526: [analyzer][StdLibraryFunctionsChecker] Add NoteTags for applied arg constraints

steakhal added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/StdLibraryFunctionsChecker.cpp:836-837 + NewState, NewNode, + C.getNoteTag([Msg](PathSensitiveBugReport &BR, + llvm::raw_ostream &OS) { OS << Msg; }));

[PATCH] D101389: [clang][CodeGen] Fix address space for sret

yaxunl abandoned this revision. yaxunl added a comment. In D101389#2724757 , @rjmccall wrote: > In D101389#2724700 , @yaxunl wrote: > >> In D101389#2724636 , @rjmccall >>

[PATCH] D99484: Use `GNUInstallDirs` to support custom installation dirs.

Ericson2314 added inline comments. Comment at: polly/cmake/CMakeLists.txt:104-110 +set(tgt_prefix "${CMAKE_INSTALL_FULL_BINDIR}/") else() -set(tgt_prefix "lib/") +set(tgt_prefix "${CMAKE_INSTALL_FULL_LIBDIR}/") endif() - set(tgt_path "${CMAKE_INSTALL_PREFIX}

[PATCH] D101168: [C++4OpenCL] Add clang extension for non-portable kernel parameters

olestrohm updated this revision to Diff 341526. olestrohm set the repository for this revision to rG LLVM Github Monorepo. olestrohm added a comment. Added a link to the C++ for OpenCL specification and cleaned up the text according to the requests. Repository: rG LLVM Github Monorepo CHANGE

[PATCH] D101479: [Driver] Support libc++ in MSVC

thakis added a comment. can you add a test for the linker invocation too? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101479/new/ https://reviews.llvm.org/D101479 ___ cfe-commits mailing list cfe-commi

[PATCH] D99488: [SYCL][Doc] Add design document for SYCL mode

Anastasia added a comment. In D99488#2725501 , @bader wrote: > In D99488#2725435 , @Anastasia wrote: > >> If I build docs now I get the following output: >> >> llvm-project/build-doc/tools/clang/docs/SYCLSupport.r

[PATCH] D91949: [clang-format] Add BeforeStructInitialization option in BraceWrapping configuration

anastasiia_lukianenko updated this revision to Diff 341537. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91949/new/ https://reviews.llvm.org/D91949 Files: clang/docs/ClangFormatStyleOptions.rst clang/include/clang/Format/Format.h clang/lib/Format/Format.cpp clang/lib/Format/Unwr

[PATCH] D101542: [NFC] Refactor ExecuteAssembler in cc1as_main.cpp

scott.linder created this revision. scott.linder requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Introduce an extra scope (another static function) to replace calls to `unique_ptr::reset` with implicit destructors via RAII. Repository:

[PATCH] D101542: [NFC] Refactor ExecuteAssembler in cc1as_main.cpp

scott.linder added reviewers: MaskRay, dineshkb-amd, aykevl. scott.linder added a comment. I'm not sure who else to add, please feel free to update the reviewers Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101542/new/ https://reviews.llvm.org/D10

[PATCH] D101462: Make it possible for targets to define their own MCObjectFileInfo

MaskRay added a comment. The `InitMCObjectFileInfo` refactoring is good, but I'll suggest `initMCObjectFileInfo(MCContext &ctx, bool PIC, bool LargeCodeModel)` (change the function name to `lowerCase`, and reorder MCContext before bool arguments). The refactoring adding `Triple` to MCContext::M

[PATCH] D101542: [NFC] Refactor ExecuteAssembler in cc1as_main.cpp

MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land. Seems reasonable. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101542/new/ https://reviews.llvm.org/D101542

[PATCH] D99432: [OPENMP]Fix PR48851: the locals are not globalized in SPMD mode.

estewart08 added a comment. In reference to https://bugs.llvm.org/show_bug.cgi?id=48851, I do not see how this helps SPMD mode with team privatization of declarations in-between target teams and parallel regions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://review

[PATCH] D99432: [OPENMP]Fix PR48851: the locals are not globalized in SPMD mode.

ABataev added a comment. In D99432#2726019 , @estewart08 wrote: > In reference to https://bugs.llvm.org/show_bug.cgi?id=48851, I do not see how > this helps SPMD mode with team privatization of declarations in-between > target teams and parallel regions.

[PATCH] D99432: [OPENMP]Fix PR48851: the locals are not globalized in SPMD mode.

estewart08 added a comment. In D99432#2726025 , @ABataev wrote: > In D99432#2726019 , @estewart08 > wrote: > >> In reference to https://bugs.llvm.org/show_bug.cgi?id=48851, I do not see >> how this helps SPMD mode

[PATCH] D99432: [OPENMP]Fix PR48851: the locals are not globalized in SPMD mode.

ABataev added a comment. In D99432#2726050 , @estewart08 wrote: > In D99432#2726025 , @ABataev wrote: > >> In D99432#2726019 , @estewart08 >> wrote: >> >>> In reference to h

[PATCH] D101549: [Doc] Fix sphynx warnings about wrong code-block format

bader created this revision. bader added a reviewer: Anastasia. Herald added a subscriber: ebevhan. bader requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D101549 Files: clan

[PATCH] D95976: [OpenMP] Simplify offloading parallel call codegen

ggeorgakoudis added a comment. In D95976#2725027 , @protze.joachim wrote: > Please update the test with a NFC commit. Thanks, @protze.joachim. The changes look good. I'll get that NFC commit in soon-ish, unless you would like to take over. Repository:

[PATCH] D99488: [SYCL][Doc] Add design document for SYCL mode

bader added a comment. Thanks! I've uploaded this version to https://reviews.llvm.org/D101549. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99488/new/ https://reviews.llvm.org/D99488 ___ cfe-commits mai

[PATCH] D97915: [Coroutines] Handle overaligned frame allocation

ychen planned changes to this revision. ychen added a comment. Found a bug. Will fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97915/new/ https://reviews.llvm.org/D97915 ___ cfe-commits mailing list

[PATCH] D101551: [CMake] Set correct CXX_FLAGS for relative-vtables variants

phosek created this revision. phosek added a reviewer: leonardchan. Herald added a subscriber: mgorny. phosek requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. We overrite CXX_FLAGS to enable relative vtables, but doing so overwrites generic

[PATCH] D101551: [CMake] Set correct CXX_FLAGS for relative-vtables variants

phosek updated this revision to Diff 341570. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101551/new/ https://reviews.llvm.org/D101551 Files: clang/cmake/caches/Fuchsia-stage2.cmake Index: clang/cmake/caches/Fuchsia-stage2.cmake ===

[clang] ba63124 - [CMake] Set correct CXX_FLAGS for relative-vtables variants

Author: Petr Hosek Date: 2021-04-29T10:34:37-07:00 New Revision: ba631240ae9ccbf70f57c0ba22cf2dd5a3592da2 URL: https://github.com/llvm/llvm-project/commit/ba631240ae9ccbf70f57c0ba22cf2dd5a3592da2 DIFF: https://github.com/llvm/llvm-project/commit/ba631240ae9ccbf70f57c0ba22cf2dd5a3592da2.diff LO

[PATCH] D101551: [CMake] Set correct CXX_FLAGS for relative-vtables variants

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGba631240ae9c: [CMake] Set correct CXX_FLAGS for relative-vtables variants (authored by phosek). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D99543: [clang-tidy] Allow opt-in or out of some commonly occuring patterns in NarrowingConversionsCheck.

ymandel accepted this revision. ymandel added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/docs/clang-tidy/checks/cppcoreguidelines-narrowing-conversions.rst:51 + + Narrowing conversions from any type in this semicolon separat

[PATCH] D99599: [NewPM] Add an option to dump pass structure

haowei added a comment. We are seeing test failures in our builders with message after your patch. Could you take a look and fix this unit test please? FAIL: Clang :: Driver/debug-pass-structure.c (6773 of 27894) TEST 'Clang :: Driver/debug-pass-structure.c' FAILED

[PATCH] D99599: [NewPM] Add an option to dump pass structure

evgeny777 added a comment. @haowei What are LLVM configuration options? Also please send output from /opt/s/w/ir/x/w/staging/llvm_build/bin/clang -flegacy-pass-manager -fdebug-pass-structure -O0 -S -emit-llvm /opt/s/w/ir/x/w/llvm-project/clang/test/Driver/debug-pass-structure.c -o /dev/null

[PATCH] D101554: [clangd] Add support for the `defaultLibrary` semantic token modifier

dgoldman created this revision. dgoldman added reviewers: sammccall, kadircet. Herald added subscribers: usaxena95, jfb, arphaman. dgoldman requested review of this revision. Herald added subscribers: cfe-commits, MaskRay, ilya-biryukov. Herald added a project: clang-tools-extra. This allows us to

[PATCH] D98300: [TLS] Initialization functions of dynamic TLS variables cannot be in a comdat (fix for PR48030)

wolfgangp added a comment. ping... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D98300/new/ https://reviews.llvm.org/D98300 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D99599: [NewPM] Add an option to dump pass structure

evgeny777 added a comment. Hm ... I see BarrierNoop pass being added before `annotation-remarks`. Why's that? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99599/new/ https://reviews.llvm.org/D99599 ___

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

delcypher added a comment. In D101491#2725348 , @jansvoboda11 wrote: > In D101491#2724025 , @delcypher > wrote: > >> @jansvoboda11 Should I use the the alias feature so that the old >> `-fsanitize-address-destr

[PATCH] D99432: [OPENMP]Fix PR48851: the locals are not globalized in SPMD mode.

estewart08 added a comment. In D99432#2726060 , @ABataev wrote: > In D99432#2726050 , @estewart08 > wrote: > >> In D99432#2726025 , @ABataev wrote: >> >>> In D99432#2726019

[PATCH] D99599: [NewPM] Add an option to dump pass structure

haowei added a comment. In D99599#2726252 , @evgeny777 wrote: > @haowei What are LLVM configuration options? Also please send output from > > /opt/s/w/ir/x/w/staging/llvm_build/bin/clang -flegacy-pass-manager > -fdebug-pass-structure -O0 -S -emit-llvm

[PATCH] D99432: [OPENMP]Fix PR48851: the locals are not globalized in SPMD mode.

ABataev added a comment. In D99432#2726337 , @estewart08 wrote: > In D99432#2726060 , @ABataev wrote: > >> In D99432#2726050 , @estewart08 >> wrote: >> >>> In D99432#2726025

[PATCH] D101503: [OpenMPIRBuilder] Add createOffloadMaptypes and createOffloadMapnames functions

clementval updated this revision to Diff 341589. clementval added a comment. Fix global variable names. Separator comes from OpenMP runtime information. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101503/new/ https://reviews.llvm.org/D101503 Fil

[PATCH] D101352: [DOCS] Removed inconsistency in clang vs Clang usage in docs (c vs C)

jrtc27 added a comment. I'm not convinced by these changes? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101352/new/ https://reviews.llvm.org/D101352 ___ cfe-commits mailing list cfe-commits@lists.llvm.

[PATCH] D97915: [Coroutines] Handle overaligned frame allocation

ychen updated this revision to Diff 341591. ychen added a comment. - fix a bug. ready for review. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97915/new/ https://reviews.llvm.org/D97915 Files: clang/lib/CodeGen/CGBuiltin.cpp clang/lib/Code

[PATCH] D101479: [Driver] Support libc++ in MSVC

rnk added a comment. The test doesn't seem to pass on Jenkins. Maybe we need a REQUIRES line or a fake sysroot or something for the test. Comment at: clang/lib/Driver/ToolChains/MSVC.cpp:1332 + + auto AddLibcxxIncludePath = [&](StringRef Path) { +std::string Version = det

[PATCH] D97915: [Coroutines] Handle overaligned frame allocation

ychen updated this revision to Diff 341592. ychen added a comment. - fix typo Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97915/new/ https://reviews.llvm.org/D97915 Files: clang/lib/CodeGen/CGBuiltin.cpp clang/lib/CodeGen/CGCoroutine.cpp c

[PATCH] D70094: [clang-tidy] new altera ID dependent backward branch check

ffrankies marked an inline comment as done. ffrankies added a comment. @Eugene.Zelenko @aaron.ballman Are there any more changes that need to be made to this check or comments that need to be addressed? Comment at: clang-tools-extra/clang-tidy/altera/IdDependentBackwardBranch

[PATCH] D97915: [Coroutines] Handle overaligned frame allocation

ychen added a comment. For coroutine `f0` in `test/CodeGenCoroutines/coro-alloc.cpp` The allocation looks like this: ; Function Attrs: noinline nounwind optnone mustprogress define dso_local void @f0() #0 { entry: %0 = alloca %struct.global_new_delete_tag, align 1 %1 = alloca %stru

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

delcypher updated this revision to Diff 341596. delcypher added a comment. Rename def too. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101491/new/ https://reviews.llvm.org/D101491 Files: clang/docs/ClangCommandLineReference.rst clang/include

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

delcypher added a comment. @jansvoboda11 I'm going to land this patch as is (with your nit fixed). If you would like me to add an alias please follow up with me and I can put up a patch to do that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101

[PATCH] D101491: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG2d42b2ee7baf: [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix. (authored by delcypher). Repository: rG LLVM Github

[clang] 2d42b2e - [ASan] Rename `-fsanitize-address-destructor-kind=` to drop the `-kind` suffix.

Author: Dan Liew Date: 2021-04-29T11:55:42-07:00 New Revision: 2d42b2ee7bafe76d6b2c792b73f7371cb1cf8d94 URL: https://github.com/llvm/llvm-project/commit/2d42b2ee7bafe76d6b2c792b73f7371cb1cf8d94 DIFF: https://github.com/llvm/llvm-project/commit/2d42b2ee7bafe76d6b2c792b73f7371cb1cf8d94.diff LOG:

[PATCH] D101561: [Prototype] Introduce attribute for ignoring C++ ABI

aeubanks created this revision. Herald added a reviewer: aaron.ballman. aeubanks requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Repository: rG LLVM Github Monorepo https://reviews.llvm.org/D101561 Files: clang/include/clang/Basic/Att

[PATCH] D91949: [clang-format] Add BeforeStructInitialization option in BraceWrapping configuration

HazardyKnusperkeks added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:2705 } - if (FormatTok->Tok.is(tok::l_brace)) { + if (FormatTok->Tok.is(tok::equal)) { +nextToken(); MyDeveloperDay wrote: > shouldn't this be > > ```FormatTo

[PATCH] D99599: [NewPM] Add an option to dump pass structure

haowei added a comment. Output from `clang -flegacy-pass-manager -fdebug-pass-structure -O0 -S -emit-llvm /opt/s/w/ir/x/w/llvm-project/clang/test/Driver/debug-pass-structure.c -o /dev/null 2>&1`: Pass Arguments: -tti -targetlibinfo Target Transform Information Target Library Information

[PATCH] D99432: [OPENMP]Fix PR48851: the locals are not globalized in SPMD mode.

estewart08 added a comment. In D99432#2726391 , @ABataev wrote: > In D99432#2726337 , @estewart08 > wrote: > >> In D99432#2726060 , @ABataev wrote: >> >>> In D99432#2726050

[PATCH] D99381: [compiler-rt][hwasan] Add Fuchsia-specific sanitizer platform limits

leonardchan updated this revision to Diff 341610. leonardchan marked an inline comment as done. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99381/new/ https://reviews.llvm.org/D99381 Files: compiler-rt/lib/hwasan/hwasan_interceptors.cpp compil

[PATCH] D99381: [compiler-rt][hwasan] Add Fuchsia-specific sanitizer platform limits

leonardchan added a comment. In D99381#2723563 , @vitalybuka wrote: > In D99381#2721619 , @phosek wrote: > >> What I think @vitalybuka meant is keeping >> `sanitizer_platform_limits_fuchsia.h` as you had it, but in

[PATCH] D101564: [OpenMP] Fix second debug name from map clause

jdenny created this revision. jdenny added reviewers: jhuber6, jdoerfert. Herald added subscribers: guansong, yaxunl. jdenny requested review of this revision. Herald added a subscriber: sstefan1. Herald added a project: clang. This patch fixes a bug from D89802 .

[PATCH] D101564: [OpenMP] Fix second debug name from map clause

jhuber6 accepted this revision. jhuber6 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/D101564/new/ https://reviews.llvm.org/D101564

[PATCH] D101566: Let -Wweak-template-vtables warn on implicit instantiations

aaronpuchert created this revision. aaronpuchert added reviewers: dblaikie, rsmith. aaronpuchert requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Implicit instantiations of template classes with virtual methods might cause the vtable (and al

  1   2   >