[PATCH] D100953: clang-format: [JS] do not wrap after `asserts`

h-joo added inline comments. Comment at: clang/lib/Format/TokenAnnotator.cpp:3856 return false; // Otherwise automatic semicolon insertion would trigger. +if (NonComment && NonComment->is(tok::identifier) && +NonComment->TokenText == "asserts")

[clang] 70ae843 - clang-format: [JS] do not wrap after `asserts`

Author: Martin Probst Date: 2021-04-21T16:33:55+02:00 New Revision: 70ae843d99808b19247e968507c0019225597066 URL: https://github.com/llvm/llvm-project/commit/70ae843d99808b19247e968507c0019225597066 DIFF: https://github.com/llvm/llvm-project/commit/70ae843d99808b19247e968507c0019225597066.diff

[PATCH] D100953: clang-format: [JS] do not wrap after `asserts`

This revision was automatically updated to reflect the committed changes. Closed by commit rG70ae843d9980: clang-format: [JS] do not wrap after `asserts` (authored by mprobst). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100953/new/ https://review

[PATCH] D99949: [AMDGPU][OpenMP] Add amdgpu-arch tool to list AMD GPUs installed

pdhaliwal updated this revision to Diff 339235. pdhaliwal added a comment. Replaced the return commands in test scripts with exit command. It seems like return is handled bit differently on fedora/rhel machines. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.l

[PATCH] D96524: [OpenCL] Add support of OpenCL C 3.0 __opencl_c_fp64

azabaznov added a comment. In D96524#2692376 , @Anastasia wrote: > In D96524#2691428 , @azabaznov wrote: > >> My main idea was to provide an interface which will not make users to >> specify `-cl-ext=+__opencl_c_fp

[PATCH] D96524: [OpenCL] Add support of OpenCL C 3.0 __opencl_c_fp64

Anastasia added a comment. In D96524#2705192 , @azabaznov wrote: > In D96524#2692376 , @Anastasia wrote: > >> In D96524#2691428 , @azabaznov >> wrote: >> >>> My main idea wa

[PATCH] D100778: [clang-format] Prevent extraneous space insertion in bitshift operators

penagos added a comment. Additionally; barring any other feedback, I'll need someone to land this change as I do not have commit access. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100778/new/ https://reviews.llvm.org/D100778 __

[PATCH] D100756: [llvm-rc] [4/4] Add a GNU windres-like frontend to llvm-rc

aganea added inline comments. Comment at: llvm/tools/llvm-rc/llvm-rc.cpp:259 -} // anonymous namespace +static bool consume_back_lower(StringRef &S, const char *str) { + if (!S.endswith_lower(str)) `s/str/Str/` Comment at: llvm/tools/llvm-rc

[PATCH] D100673: [OPENMP]Fix PR49698: OpenMP declare mapper causes segmentation fault.

ABataev added inline comments. Comment at: openmp/libomptarget/test/mapping/declare_mapper_nested_default_mappers_complex_structure.cpp:123-126 + assert(outer[1].arr[1].arr[0].data1 == 11 && + outer[1].arr[1].arr[0].data2 == 22 && + outer[1].arr[1].arr[1].data1

[PATCH] D100860: [C++4OpenCL] Add missing OpenCL specific diagnostics in templates

olestrohm updated this revision to Diff 339251. olestrohm added a comment. Renamed test as suggested. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100860/new/ https://reviews.llvm.org/D100860 Files: clang/lib/Sema/SemaDecl.cpp clang/test/SemaOpenCL/clk_event_t.cl clang/test/Sema

[PATCH] D99233: [HIP] Add option --gpu-inline-threshold

yaxunl added a comment. In D99233#2656446 , @tra wrote: > I'm concerned that if we make it a top-level option, it's likely to be > cargo-culted and (mis)used as a sledgehammer in cases where it's not needed. > I think the option should remain hidden. > >

[PATCH] D100756: [llvm-rc] [4/4] Add a GNU windres-like frontend to llvm-rc

mstorsjo added inline comments. Comment at: llvm/tools/llvm-rc/llvm-rc.cpp:259 -} // anonymous namespace +static bool consume_back_lower(StringRef &S, const char *str) { + if (!S.endswith_lower(str)) aganea wrote: > `s/str/Str/` Thanks, will fix. ===

[PATCH] D69498: IR: Invert convergent attribute handling

sameerds added a comment. I realize now that what @foad says above puts the idea in a clearer context. Essentially, any check for isConvergent() isn't happening in a vacuum. It is relevant only in the presence of divergent control flow, which in turn is relevant only when the target has diverge

[PATCH] D100942: [clang][deps] Include "-cc1" in the arguments

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/D100942/new/ https://reviews.llvm.org/D100942

[PATCH] D100756: [llvm-rc] [4/4] Add a GNU windres-like frontend to llvm-rc

mstorsjo updated this revision to Diff 339263. mstorsjo added a comment. Updated the capitalization of one variable Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100756/new/ https://reviews.llvm.org/D100756 Files: clang/test/CMakeLists.txt cla

[PATCH] D100819: [RISCV] Implement the vneg.v builtin.

craig.topper added inline comments. Comment at: clang/utils/TableGen/RISCVVEmitter.cpp:1048 + return Cmp < 0; + // Some mask intrinsics use the same IRName as unmasked. + // Sort the unmasked intrinsics first.

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

ggeorgakoudis updated this revision to Diff 339265. ggeorgakoudis added a comment. Add tests, reduce microtask cases to avoid stack problems Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95976/new/ https://reviews.llvm.org/D95976 Files: clang/li

[PATCH] D100581: [Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable

mbenfield updated this revision to Diff 339275. mbenfield added a comment. Herald added subscribers: frasercrmck, luismarques, apazos, sameer.abuasal, pengfei, s.egerton, Jim, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, niosHD, sabuasal, simoncook, joh

[PATCH] D100581: [Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable

mbenfield added a comment. In D100581#2700083 , @xbolva00 wrote: > So I think we should put UnusedButSetParameter under -Wextra > > So I would just put UnusedButSetVariable under -Wunused (and -Wunused is part > of -Wall): > > WDYT? Sounds good; I've do

[PATCH] D100968: Update shebang for clang-format-diff script to python3.

PaulkaToast created this revision. PaulkaToast added reviewers: MaskRay, DavidSpickett. PaulkaToast requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. On newer Linux distributions that don't have python2, the clang-format-diff script and cons

[PATCH] D100756: [llvm-rc] [4/4] Add a GNU windres-like frontend to llvm-rc

aganea added inline comments. Comment at: llvm/tools/llvm-rc/llvm-rc.cpp:300 + +std::string unescape(StringRef S) { + std::string Out; mstorsjo wrote: > aganea wrote: > > I would also need this function in D43002 (see unescapeSlashes), do you > > think we can m

[PATCH] D100874: [OpenMP] Use irbuilder as default for masked and master construct

cchen added a comment. > what assertion? $ ":" "RUN: at line 1" $ "/mnt/disks/ssd0/agent/llvm-project/build/./bin/clang" "-fopenmp" "-pthread" "-fno-experimental-isel" "-I" "/mnt/disks/ssd0/agent/llvm-project/openmp/runtime/test" "-I" "/mnt/disks/ssd0/agent/llvm-project/build/projects/op

[PATCH] D100968: Update shebang for clang-format-diff script to python3.

MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land. > On newer Linux distributions Different distributions have different strategies migrating the /usr/bin/python symlink. Debian and its derivatives provide python-is-python2 and python-is-py

[PATCH] D100968: Update shebang for clang-format-diff script to python3.

PaulkaToast added a comment. In D100968#2705598 , @MaskRay wrote: >> On newer Linux distributions > > Different distributions have different strategies migrating the > /usr/bin/python symlink. Debian and its derivatives provide python-is-python2 > and p

[PATCH] D100972: [clang-tidy] cppcoreguidelines-avoid-non-const-global-variables: add fixes to checks

mgartmann created this revision. mgartmann added reviewers: aaron.ballman, njames93, alexfh. mgartmann added a project: clang-tools-extra. Herald added subscribers: shchenz, kbarton, xazax.hun, nemanjai. mgartmann requested review of this revision. I added fixes to the existing checks of `cppcore

[PATCH] D100955: [-Wcalled-once] Do not run analysis on Obj-C++

NoQ accepted this revision. NoQ added a comment. This revision is now accepted and ready to land. Ok then! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100955/new/ https://reviews.llvm.org/D100955 ___ c

[PATCH] D45639: [Driver] Support default libc++ library location on Darwin

ldionne added a comment. In D45639#2703913 , @JDevlieghere wrote: > This breaks `TestAppleSimulatorOSType.py ` on GreenDragon. First failed > build: http://green.lab.llvm.org/green/view/LLDB/job/lldb-cmake/31346/ > [...] > > Based on your description abo

[PATCH] D100276: [clang] p1099 using enum part 1

urnathan updated this revision to Diff 339293. urnathan added a comment. Updated to fix formatting etc (pretty sure I got clang-format to work this time). I changed the pre-c++20 behaviour to be a warning along the lines you suggested. Trying to move the qualifier checking until after generati

[PATCH] D100976: [OpenCL] Simplify use of C11 atomic types

Anastasia created this revision. Anastasia added reviewers: svenvh, azabaznov. Herald added subscribers: ebevhan, jfb, yaxunl. Anastasia requested review of this revision. Clang's implementation of the extension pragma is broken because it doesn't respect the following requirement from OpenCL Ext

[PATCH] D100581: [Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable

xbolva00 added a comment. Thanks, looks really good. Meanwhile I collected some interesting testcases from gcc bugzilla (reported as false positives), please try: https://godbolt.org/z/747ndKEvd - No "unused-but-set" warnings expected. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[clang] 4cbe488 - [lsan][docs] Clarify supported platforms

Author: Fangrui Song Date: 2021-04-21T10:27:55-07:00 New Revision: 4cbe4881882edd64a7ff53da52970c4b6333de1c URL: https://github.com/llvm/llvm-project/commit/4cbe4881882edd64a7ff53da52970c4b6333de1c DIFF: https://github.com/llvm/llvm-project/commit/4cbe4881882edd64a7ff53da52970c4b6333de1c.diff

[PATCH] D100907: [lsan][docs] Clarify supported platforms

This revision was automatically updated to reflect the committed changes. Closed by commit rG4cbe4881882e: [lsan][docs] Clarify supported platforms (authored by MaskRay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100907/new/ https://reviews.llvm

[PATCH] D100980: [OpenCL] Allow use of double type without extension pragma

Anastasia created this revision. Anastasia added reviewers: svenvh, azabaznov. Herald added subscribers: ebevhan, yaxunl. Anastasia requested review of this revision. OpenCL specification doesn't require the pragma for the uses of `double` type when it is supported by the targets. The wording in

[PATCH] D100673: [OPENMP]Fix PR49698: OpenMP declare mapper causes segmentation fault.

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG079884225a55: [OPENMP]Fix PR49698: OpenMP declare mapper causes segmentation fault. (authored by ABataev). Repository: rG LLVM Github Monorepo CH

[clang] 0798842 - [OPENMP]Fix PR49698: OpenMP declare mapper causes segmentation fault.

Author: Alexey Bataev Date: 2021-04-21T10:38:31-07:00 New Revision: 079884225a5571f4caf1a9c71b6748db8192e383 URL: https://github.com/llvm/llvm-project/commit/079884225a5571f4caf1a9c71b6748db8192e383 DIFF: https://github.com/llvm/llvm-project/commit/079884225a5571f4caf1a9c71b6748db8192e383.diff

[PATCH] D100611: [RISCV] Add new attribute __clang_riscv_builtin_alias for intrinsics.

aaron.ballman added inline comments. Comment at: clang/include/clang/Basic/AttrDocs.td:2147-2150 +This attribute can only be used to set up the aliases for certain RISC-V +C intrinsic functions; it is intended for use only inside ``riscv_*.h`` +and is not a general mechanism for

[PATCH] D99983: Provide TreeTransform::TransformAttr the transformed statement; NFC

aaron.ballman updated this revision to Diff 339319. aaron.ballman added a comment. Now that [[clang::musttail]] has landed, I can show the refactoring I had in mind by implementing it myself. WDYT of this approach? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99983/new/ https://review

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

Meinersbur accepted this revision. Meinersbur added a comment. This revision is now accepted and ready to land. This test seem to pass on Windows now. Please still fix the clang-format remarks, such as going over 80 characters on a line. Comment at: clang/lib/CodeGen/CGOpenMPR

[PATCH] D45639: [Driver] Support default libc++ library location on Darwin

phosek added a comment. In D45639#2705702 , @ldionne wrote: > In D45639#2703913 , @JDevlieghere > wrote: > >> This breaks `TestAppleSimulatorOSType.py ` on GreenDragon. First failed >> build: http://green.lab.llvm

[PATCH] D100981: Delete le32/le64 targets

MaskRay created this revision. MaskRay added a reviewer: dschuff. Herald added subscribers: dexonsmith, jfb, hiraditya, mgorny. MaskRay requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, aheejin. Herald added projects: clang, LLVM. They are unused now. Note: N

[PATCH] D99233: [HIP] Add option --gpu-inline-threshold

tra added a comment. > The planned new option for offloading will be a more generic solution, > however, I expect it will take time to develop and be adopted. Agreed. OK, let's use a hidden option until we have a better way of dealing with this. Comment at: clang/include/cla

[PATCH] D100834: Bug 49739 - [Matrix] Support #pragma clang fp

effective-light updated this revision to Diff 339325. effective-light added a comment. Add a test for compound operations Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100834/new/ https://reviews.llvm.org/D100834 Files: clang/lib/CodeGen/CGExprS

[PATCH] D100834: Bug 49739 - [Matrix] Support #pragma clang fp

effective-light added a comment. In D100834#2704330 , @fhahn wrote: > In D100834#2702550 , @kpn wrote: > >> I don't know the matrix implementation so I can't swear this hits every >> place needed, but the uses of

[PATCH] D45639: [Driver] Support default libc++ library location on Darwin

JDevlieghere added a comment. In D45639#2705919 , @phosek wrote: > In D45639#2705702 , @ldionne wrote: > >> In D45639#2703913 , @JDevlieghere >> wrote: >> >>> This breaks `T

[PATCH] D100756: [llvm-rc] [4/4] Add a GNU windres-like frontend to llvm-rc

mstorsjo marked an inline comment as done. mstorsjo added inline comments. Comment at: llvm/tools/llvm-rc/llvm-rc.cpp:300 + +std::string unescape(StringRef S) { + std::string Out; aganea wrote: > mstorsjo wrote: > > aganea wrote: > > > I would also need this fun

[PATCH] D100118: [clang] RFC Support new builtin __arithmetic_fence to control floating point optimization, and new clang option fprotect-parens

mibintc added inline comments. Comment at: clang/lib/CodeGen/CGBuiltin.cpp:2844 + return RValue::get( + Builder.CreateArithmeticFence(ArgValue, ConvertType(ArgType))); +return RValue::get(ArgValue); mibintc wrote: > mibintc wrote: > > kpn wrote:

[PATCH] D100983: [OpenCL] Fix optional image types

Anastasia created this revision. Anastasia added reviewers: svenvh, azabaznov, yaxunl. Herald added a subscriber: ebevhan. Anastasia requested review of this revision. This change allows the use of identifiers for image types from `cl_khr_gl_msaa_sharing` freely in the kernel code if the extensio

[PATCH] D91950: [clang-format] Add BreakBeforeInlineASMColon configuration

HazardyKnusperkeks requested changes to this revision. HazardyKnusperkeks added a comment. This revision now requires changes to proceed. Looks good, but please fix the clang-format notes. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91950/new/ https://reviews.llvm.org/D91950

[PATCH] D99233: [HIP] Add option --gpu-inline-threshold

yaxunl marked an inline comment as done. yaxunl added inline comments. Comment at: clang/include/clang/Driver/Options.td:954 +def gpu_inline_threshold_EQ : Joined<["--"], "gpu-inline-threshold=">, + Flags<[CC1Option]>, + HelpText<"Inline threshold for device compilation for HIP

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

Meinersbur added a comment. What is the status of this? The parent D89671 has been accepted, but not committed. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D91054/new/ https://reviews.llvm.org/D91054 ___

[PATCH] D100124: [Clang][NVPTX] Add NVPTX intrinsics and builtins for CUDA PTX redux.sync instructions

tra added a comment. Do you know if any existing code already uses the `__nvvm_*` builtins for `cp.async`? In other words, does nvcc provide them already or is it something we're free to name as we wish? I do not see any relevant intrinsics mentioned in NVVM IR spec: https://docs.nvidia.com/cud

[PATCH] D100984: [OpenCL] Remove the need for subgroupd extension pragma in enqueue kernel builtins

Anastasia created this revision. Anastasia added reviewers: svenvh, azabaznov, mantognini. Herald added subscribers: ebevhan, yaxunl. Anastasia requested review of this revision. There is no extension pragma requirement in the spec for these functions and all other builtin functions are available

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

ggeorgakoudis updated this revision to Diff 339334. ggeorgakoudis added a comment. Fix clang-format Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95976/new/ https://reviews.llvm.org/D95976 Files: clang/lib/CodeGen/CGOpenMPRuntimeGPU.cpp clang/

[PATCH] D100985: [OpenCL] Remove pragma requirement for functions from Arm dot extension

Anastasia created this revision. Anastasia added reviewers: svenvh, mantognini. Herald added subscribers: ebevhan, kristof.beyls, yaxunl. Anastasia requested review of this revision. This removed the pointless need for extension pragma since it doesn't disable anything properly and it doesn't nee

[PATCH] D97183: [analyzer] Add NoteTag for smart-ptr get()

RedDocMD added a comment. @NoQ? (I actually should remove some extra includes and extra member fields) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97183/new/ https://reviews.llvm.org/D97183 ___ cfe-com

[PATCH] D99924: [clang-tidy] Avoid bugprone-macro-parentheses warnings after goto argument

aaron.ballman added a subscriber: cfe-commits. aaron.ballman added a comment. Adding the CFE mailing list, which was not on the review previously. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99924/new/ https://reviews.llvm.org/D99924 ___ cf

[PATCH] D100000: [clang] WIP: Implement simpler alternative to two-phase lookup for NRVO

mizvekov updated this revision to Diff 339349. mizvekov added a comment. . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D10/new/ https://reviews.llvm.org/D10 Files: clang/include/clang/AST/Expr.h clang/include/clang/Basic/DiagnosticSem

[PATCH] D100991: Fix parsing of vector keyword in presence of conflicting uses.

jamieschmeiser created this revision. jamieschmeiser added reviewers: rsmith, saar.raz. jamieschmeiser requested review of this revision. Herald added a project: clang. When vector is found as a type or non-type id, check if it is really the altivec vector token. Call TryAltiVecVectorToken when

[PATCH] D98798: Produce warning for performing pointer arithmetic on a null pointer.

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

[PATCH] D99233: [HIP] Add option --gpu-inline-threshold

yaxunl updated this revision to Diff 339353. yaxunl marked an inline comment as done. yaxunl edited the summary of this revision. yaxunl added a comment. revised by Artem's comments. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99233/new/ https://reviews.llvm.org/D99233 Files: clang

[PATCH] D93031: Enable fexec-charset option

abhina.sreeskantharajan updated this revision to Diff 339355. abhina.sreeskantharajan added a reviewer: ThePhD. abhina.sreeskantharajan added a comment. Thanks for catching that. This sets the __clang_literal_encoding__ to Opts.ExecCharset or defaults to SystemCharset. Repository: rG LLVM G

[PATCH] D98450: [clang] store an identifer instead of declref for ns_error_domain attribute

aaron.ballman added a comment. In D98450#2699970 , @arphaman wrote: > Your argument makes sense. The problem right now is that clang doesn't allow > the name to be used even when the user marks up availability correctly trying > to guard the use of the d

[PATCH] D45639: [Driver] Support default libc++ library location on Darwin

JDevlieghere added a comment. Given that these tests are macOS specific and already require a specific SDK, I'll just update them to use the compiler from the SDK instead of the just-built one. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D45639/n

[PATCH] D100466: clang-format: [JS] merge import lines.

jgorbe added a comment. Hi, one of our tests is failing because of this patch and it looks like an actual bug. clang-format now turns this file: import './a'; import {bar} from './a'; into this: barimport './a'; Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https:/

[PATCH] D99299: Normalize interaction with boolean attributes

dblaikie added inline comments. Comment at: llvm/lib/IR/Attributes.cpp:660-663 +bool AttributeImpl::getValueAsBool() const { + assert(getValueAsString().empty() || getValueAsString() == "false" || getValueAsString() == "true"); + return getValueAsString() == "true"; +} ---

[PATCH] D100581: [Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable

dblaikie added a comment. FWIW, I'd love it if we could do a full dead-store warning, which would be a superset of this. I think we have enough infrastructure in the analysis based warnings (I think the sufficiency of the infrastructure is demonstrated by the "may be used uninitialized" warning

[PATCH] D99983: Provide TreeTransform::TransformAttr the transformed statement; NFC

haberman accepted this revision. haberman added a comment. This revision is now accepted and ready to land. This seems fine to me, but I'll defer to @rsmith for final review. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D99983/new/ https://reviews.llvm.org/D99983 __

[PATCH] D99994: [CodeView] Add CodeView support for PGO debug information

Holman added a comment. Can someone help me get this checked in? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D4/new/ https://reviews.llvm.org/D4 ___ cfe-commits mailing list cfe-commits@lists.ll

[PATCH] D99233: [HIP] Add option -fgpu-inline-threshold

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5a2d78b16397: [HIP] Add option -fgpu-inline-threshold (authored by yaxunl). Herald added a project: clang. Repository: rG LLVM Github Monorepo CH

[clang] 5a2d78b - [HIP] Add option -fgpu-inline-threshold

Author: Yaxun (Sam) Liu Date: 2021-04-21T17:18:18-04:00 New Revision: 5a2d78b1639738fbde85248413c1cfd2df55683a URL: https://github.com/llvm/llvm-project/commit/5a2d78b1639738fbde85248413c1cfd2df55683a DIFF: https://github.com/llvm/llvm-project/commit/5a2d78b1639738fbde85248413c1cfd2df55683a.dif

[clang] 6331680 - Re-land "[Driver] Support default libc++ library location on Darwin"

Author: Jonas Devlieghere Date: 2021-04-21T14:22:13-07:00 New Revision: 6331680ad2ad000fdaf7e72f3c1880c7908ffa25 URL: https://github.com/llvm/llvm-project/commit/6331680ad2ad000fdaf7e72f3c1880c7908ffa25 DIFF: https://github.com/llvm/llvm-project/commit/6331680ad2ad000fdaf7e72f3c1880c7908ffa25.d

[PATCH] D45639: [Driver] Support default libc++ library location on Darwin

JDevlieghere added a comment. In D45639#2706364 , @JDevlieghere wrote: > Given that these tests are macOS specific and already require a specific SDK, > I'll just update them to use the compiler from the SDK instead of the > just-built one. Done in 5d1

[PATCH] D100581: [Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable

xbolva00 added a comment. In D100581#2706467 , @dblaikie wrote: > FWIW, I'd love it if we could do a full dead-store warning, which would be a > superset of this. I think we have enough infrastructure in the analysis based > warnings (I think the suffic

[PATCH] D100968: Update shebang for clang-format-diff script to python3.

MaskRay added a comment. Still LG:) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100968/new/ https://reviews.llvm.org/D100968 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.o

[PATCH] D100581: [Clang] -Wunused-but-set-parameter and -Wunused-but-set-variable

dblaikie added a comment. In D100581#2706554 , @xbolva00 wrote: > In D100581#2706467 , @dblaikie > wrote: > >> FWIW, I'd love it if we could do a full dead-store warning, which would be a >> superset of this. I

[PATCH] D45639: [Driver] Support default libc++ library location on Darwin

dexonsmith added a comment. > No, the bot is also meant to catch changes to libc++ breaking LLDB (or at > least making sure we update the corresponding data formatters). > Given that these tests are macOS specific and already require a specific SDK, > I'll just update them to use the compiler

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

lxfind added inline comments. Comment at: clang/docs/LanguageExtensions.rst:2689 - size_t __builtin_coro_size() + size_t __builtin_coro_size(bool alloc) void *__builtin_coro_frame() ChuanqiXu wrote: > ychen wrote: > > ChuanqiXu wrote: > > > ychen wrote: >

[PATCH] D45639: [Driver] Support default libc++ library location on Darwin

ldionne added a comment. In D45639#2706589 , @dexonsmith wrote: > I'm not sure I'm totally following, but just want to double check that the > tests won't somehow use the libc++ from the SDK instead of ToT? I guess the > test uses `-nostdinc++` or someth

[PATCH] D93031: Enable fexec-charset option

cor3ntin added a comment. >> We should use the original source form of the string literal when >> pretty-printing a `StringLiteral` or `CharacterLiteral`; there are a bunch >> of UTF-8 assumptions baked into `StmtPrinter` that will need revisiting. And >> we'll need to modify the handful of pla

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

leonardchan updated this revision to Diff 339400. This revision is now accepted and ready to land. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85802/new/ https://reviews.llvm.org/D85802 Files: clang/include/clang/AST/ASTContext.h clang/include

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

leonardchan added a comment. I got LGTM for this before, but I'll leave it open for a few days to address any new comments folks might have. Comment at: clang/lib/Frontend/CompilerInvocation.cpp:3523 +else + Opts.CXXABI = TargetCXXABI::getKind(CXXABI); + } --

[PATCH] D101000: Coverage: Document how to collect a profile without a filesystem

dexonsmith created this revision. dexonsmith added a reviewer: vsk. dexonsmith requested review of this revision. Herald added a project: clang. The profiling runtime was designed to work without static initializers or a a filesystem (see 117cf2bd1ff585f9754b5f30f5a4cfd65b230bbf and others). The n

[PATCH] D100611: [RISCV] Add new attribute __clang_riscv_builtin_alias for intrinsics.

HsiangKai added inline comments. Comment at: clang/include/clang/Basic/AttrDocs.td:2147-2150 +This attribute can only be used to set up the aliases for certain RISC-V +C intrinsic functions; it is intended for use only inside ``riscv_*.h`` +and is not a general mechanism for decl

[PATCH] D101000: Coverage: Document how to collect a profile without a filesystem

vsk added a subscriber: efriedma. vsk added a comment. Thanks for doing this! + Eli to comment on whether any workarounds documented in https://lists.llvm.org/pipermail/llvm-dev/2017-September/117156.html are still necessary. To my knowledge it's not currently necessary to explicitly pass -mll

[PATCH] D99994: [CodeView] Add CodeView support for PGO debug information

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG77357208c46a: [CodeView] Add CodeView support for PGO debug information (authored by Holman, committed by rnk). Repository: rG LLVM Github Monorep

[PATCH] D99994: [CodeView] Add CodeView support for PGO debug information

rnk added a comment. In D4#2706515 , @Holman wrote: > Can someone help me get this checked in? Sure, I went ahead and pushed it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D4/new/ https://reviews

[libunwind] f749550 - [libcxx] Stop using use c++ subdirectory for libc++ library

Author: Petr Hosek Date: 2021-04-21T15:39:03-07:00 New Revision: f749550cfe9f0bf2364abb2139835348587062ed URL: https://github.com/llvm/llvm-project/commit/f749550cfe9f0bf2364abb2139835348587062ed DIFF: https://github.com/llvm/llvm-project/commit/f749550cfe9f0bf2364abb2139835348587062ed.diff LO

[clang] f749550 - [libcxx] Stop using use c++ subdirectory for libc++ library

Author: Petr Hosek Date: 2021-04-21T15:39:03-07:00 New Revision: f749550cfe9f0bf2364abb2139835348587062ed URL: https://github.com/llvm/llvm-project/commit/f749550cfe9f0bf2364abb2139835348587062ed DIFF: https://github.com/llvm/llvm-project/commit/f749550cfe9f0bf2364abb2139835348587062ed.diff LO

[PATCH] D100869: [libcxx] Stop using use c++ subdirectory for libc++ library

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 rGf749550cfe9f: [libcxx] Stop using use c++ subdirectory for libc++

[clang] 775a948 - [IR][sanitizer] Set nounwind on module ctor/dtor, additionally set uwtable if -fasynchronous-unwind-tables

Author: Fangrui Song Date: 2021-04-21T15:58:20-07:00 New Revision: 775a9483e55efb365691bc425ed107568ad6daf8 URL: https://github.com/llvm/llvm-project/commit/775a9483e55efb365691bc425ed107568ad6daf8 DIFF: https://github.com/llvm/llvm-project/commit/775a9483e55efb365691bc425ed107568ad6daf8.diff

[PATCH] D100251: [IR][sanitizer] Set nounwind on module ctor/dtor, additionally set uwtable if -fasynchronous-unwind-tables

This revision was automatically updated to reflect the committed changes. Closed by commit rG775a9483e55e: [IR][sanitizer] Set nounwind on module ctor/dtor, additionally set uwtable if… (authored by MaskRay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.

[PATCH] D101000: Coverage: Document how to collect a profile without a filesystem

dexonsmith updated this revision to Diff 339423. dexonsmith added a comment. Incorporate feedback from @vsk: - a bit more explanatory text for the overall context - explicitly reference the static initializers CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101000/new/ https://reviews.ll

[PATCH] D101000: Coverage: Document how to collect a profile without a filesystem

dexonsmith marked an inline comment as done. dexonsmith added inline comments. Comment at: clang/docs/SourceBasedCodeCoverage.rst:368 + +In C++ files, declare these as ``extern "C"``. + vsk wrote: > I think we should leave a note about how to avoid pulling in unw

[PATCH] D100981: Delete le32/le64 targets

dschuff accepted this revision. dschuff added a comment. This revision is now accepted and ready to land. Thanks. I had heard in the past that there were some other folks who had used le32/le64 as a "generic" target (in fact that's why it's named so generically, rather than being called "pnacl"

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

lxfind added a comment. Thanks for working on this. I am still having a bit hard time understanding the solution. A few questions: 1. I assume this patch is to solve the problem where the promise object is not aligned according to its alignof annotation, right? The title/wording is a bit mislea

[PATCH] D93031: Enable fexec-charset option

joerg added a comment. "Keeping the original spelling around" would assume that the input is not using a stateful encoding. That seems worse as assumption than giving the canonical output in UTF-8 and shifting the problem to the user's editor? Repository: rG LLVM Github Monorepo CHANGES SIN

[clang] 77ac823 - Delete le32/le64 targets

Author: Fangrui Song Date: 2021-04-21T18:44:12-07:00 New Revision: 77ac823fd285973cfb3517932c09d82e6a32f46d URL: https://github.com/llvm/llvm-project/commit/77ac823fd285973cfb3517932c09d82e6a32f46d DIFF: https://github.com/llvm/llvm-project/commit/77ac823fd285973cfb3517932c09d82e6a32f46d.diff

[PATCH] D100981: Delete le32/le64 targets

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG77ac823fd285: Delete le32/le64 targets (authored by MaskRay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm

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

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa2dbfb6b72db: [OpenMP] Simplify offloading parallel call codegen (authored by ggeorgakoudis). Changed prior to commit: https://reviews.llvm.org/D9

[PATCH] D100830: [RISCV] [1/2] Add IR intrinsic for Zbp extension

craig.topper accepted this revision. craig.topper 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/D100830/new/ https://reviews.llvm.org/D100830 ___

<    1   2   3   >