[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

uabelho added a comment. In D129242#3687968 , @sunho wrote: > https://reviews.llvm.org/D130788 this is the patch fyi. Great, the patch works for us. Thanks! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D12

[PATCH] D128569: Start support for HLSL `RWBuffer`

uabelho added a comment. Hello, Compiling with gcc I get the following warning with this patch: [1832/2330] Building CXX object tools/clang/lib/Sema/CMakeFiles/obj.clangSema.dir/HLSLExternalSemaSource.cpp.o In file included from ../../clang/include/clang/Sema/ExternalSemaSource.h:15,

[PATCH] D122768: [Clang][C++20] Support capturing structured bindings in lambdas

cor3ntin updated this revision to Diff 448773. cor3ntin added a comment. Fix formatting Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122768/new/ https://reviews.llvm.org/D122768 Files: clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp

[PATCH] D130807: [InstrProf] Add the omitprofile attribute

paulkirth added a comment. Do you expect the difference between `noprofile` and `omitprofile` to be confusing to users? I can certainly see how users could be confused by what the difference is between `noprofile` and `omitprofile` ... Since what you want to communicate is "never profile"(which

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

alanphipps added a comment. In D130786#3688503 , @sunho wrote: > @alanphipps I just confirmed that the buildbot failures were fixed by the new > fix https://reviews.llvm.org/rG65c9265f4158. Could you check out if this > fixes the failure on your end too

[PATCH] D128158: [AMDGPU] Add amdgcn_sched_group_barrier builtin

uabelho added inline comments. Comment at: llvm/lib/Target/AMDGPU/AMDGPUIGroupLP.cpp:314 + +bool SchedGroup::isFull() const { + return MaxSize && Collection.size() >= *MaxSize; Compiling with gcc, I get a warning that this function is unused. I'm wondering, ther

[PATCH] D112374: [clang] Implement ElaboratedType sugaring for types written bare

rtrieu added inline comments. Comment at: clang/lib/AST/QualTypeNames.cpp:455 + if (const auto *UT = QT->getAs()) { +return getFullyQualifiedType(UT->getUnderlyingType(), Ctx, + WithGlobalNsPrefix); Moving this down here means

[clang-tools-extra] 873888c - Use is_sorted (NFC)

Author: Kazu Hirata Date: 2022-07-29T21:18:42-07:00 New Revision: 873888c179744eb0a35d4b6d69424c6adf3d08d4 URL: https://github.com/llvm/llvm-project/commit/873888c179744eb0a35d4b6d69424c6adf3d08d4 DIFF: https://github.com/llvm/llvm-project/commit/873888c179744eb0a35d4b6d69424c6adf3d08d4.diff L

[clang] e5a1ccb - Use value instead of getValue (NFC)

Author: Kazu Hirata Date: 2022-07-29T21:18:41-07:00 New Revision: e5a1ccbf254e2e44830fde3d83357c06b814f881 URL: https://github.com/llvm/llvm-project/commit/e5a1ccbf254e2e44830fde3d83357c06b814f881 DIFF: https://github.com/llvm/llvm-project/commit/e5a1ccbf254e2e44830fde3d83357c06b814f881.diff L

[clang] a948117 - [clang] Use has_value instead of value (NFC)

Author: Kazu Hirata Date: 2022-07-29T21:18:39-07:00 New Revision: a9481170888447176f3552c62ff7dca693c25b97 URL: https://github.com/llvm/llvm-project/commit/a9481170888447176f3552c62ff7dca693c25b97 DIFF: https://github.com/llvm/llvm-project/commit/a9481170888447176f3552c62ff7dca693c25b97.diff L

[clang] fb7fa27 - Preserve qualifiers when getting fully qualified type

Author: Weverything Date: 2022-07-29T19:42:54-07:00 New Revision: fb7fa27f92caf10315d3fb7def99affb71b4fe44 URL: https://github.com/llvm/llvm-project/commit/fb7fa27f92caf10315d3fb7def99affb71b4fe44 DIFF: https://github.com/llvm/llvm-project/commit/fb7fa27f92caf10315d3fb7def99affb71b4fe44.diff L

[PATCH] D130811: [Clang] Fix handling of Max from getValueRange(...) in IntExprEvaluator::VisitCastExpr(...)

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 rGa0d610516288: [Clang] Fix handling of Max from getValueRange(...)

[clang] a0d6105 - [Clang] Fix handling of Max from getValueRange(...) in IntExprEvaluator::VisitCastExpr(...)

Author: Shafik Yaghmour Date: 2022-07-29T19:17:42-07:00 New Revision: a0d61051628825e2f394240dc442ad5d81fa78bc URL: https://github.com/llvm/llvm-project/commit/a0d61051628825e2f394240dc442ad5d81fa78bc DIFF: https://github.com/llvm/llvm-project/commit/a0d61051628825e2f394240dc442ad5d81fa78bc.dif

[PATCH] D130620: Fix lack of cc1 flag in llvmcmd sections when assertions are enabled

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGafb4efd3bcc6: Fix lack of cc1 flag in llvmcmd sections when assertions are enabled (authored by aidengrossman, committed by mtrofin). Repository:

[clang] afb4efd - Fix lack of cc1 flag in llvmcmd sections when assertions are enabled

Author: Aiden Grossman Date: 2022-07-29T18:51:48-07:00 New Revision: afb4efd3bcc68ab95bf3c35183bedbdbf038356a URL: https://github.com/llvm/llvm-project/commit/afb4efd3bcc68ab95bf3c35183bedbdbf038356a DIFF: https://github.com/llvm/llvm-project/commit/afb4efd3bcc68ab95bf3c35183bedbdbf038356a.diff

[PATCH] D130620: Fix lack of cc1 flag in llvmcmd sections when assertions are enabled

mtrofin added a comment. In D130620#3686357 , @aidengrossman wrote: > Thanks for the review and your suggestions. Do you mind pushing this commit? > I don't currently have commit access to LLVM. Thanks. I can do it. Repository: rG LLVM Github Monor

[PATCH] D130811: [Clang] Fix handling of Max from getValueRange(...) in IntExprEvaluator::VisitCastExpr(...)

shafik added a comment. @thakis I created a PR for the fix to D130058 since I wanted to document this. As long the pre-commit checks don't show anything off I will land this. I added a test to also cover your case as well. CHANGES SINCE LAST ACTION https:/

[PATCH] D130811: [Clang] Fix handling of Max from getValueRange(...) in IntExprEvaluator::VisitCastExpr(...)

shafik created this revision. shafik added reviewers: erichkeane, aaron.ballman, tahonermann, thakis. Herald added a project: All. shafik requested review of this revision. This is a follow-up to D130058 to fix how we handle the `Max` value we obtain from `getVa

[PATCH] D130808: [InstrProf] Add new format for -fprofile-list=

ellis created this revision. Herald added a project: All. ellis edited the summary of this revision. ellis added a reviewer: phosek. ellis edited the summary of this revision. ellis added a reviewer: davidxl. ellis published this revision for review. Herald added a project: clang. Herald added a su

[PATCH] D130807: [InstrProf] Add the omitprofile attribute

ellis created this revision. Herald added subscribers: Enna1, wenlei, jdoerfert, hiraditya. Herald added a project: All. ellis edited the summary of this revision. ellis added reviewers: phosek, davidxl. ellis published this revision for review. Herald added projects: clang, LLVM. Herald added subs

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

sunho added a comment. @alanphipps I just confirmed that the buildbot failures were fixed by the new fix https://reviews.llvm.org/rG65c9265f4158. Could you check out if this fixes the failure on your end too? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.ll

[clang] 32f59b3 - [clang-repl] Add missing link component.

Author: Sunho Kim Date: 2022-07-30T08:00:15+09:00 New Revision: 32f59b34b663c48c217047d4963ad3d1a2378c23 URL: https://github.com/llvm/llvm-project/commit/32f59b34b663c48c217047d4963ad3d1a2378c23 DIFF: https://github.com/llvm/llvm-project/commit/32f59b34b663c48c217047d4963ad3d1a2378c23.diff LOG

[PATCH] D128757: [Driver][test] Add -fuse-ld= option tests for NetBSD

MaskRay added a comment. Windows is very different from all other OSes. Having a fuse-ld-windows.c may be fine. Again, NetBSD shares many characteristics with other OSes and having a test for all ~10 ELF OSes Clang supports seems too much. Repository: rG LLVM Github Monorepo CHANGES SINCE L

[PATCH] D129954: [CodeGen][inlineasm] assume the flag output of inline asm is boolean value

ychen marked 2 inline comments as done. ychen added a comment. In D129954#3686780 , @nikic wrote: > LGTM. After some further consideration, implementing this properly in LLVM > would probably take more effort than is worthwhile (especially as this is >

[PATCH] D129954: [CodeGen][inlineasm] assume the flag output of inline asm is boolean value

ychen updated this revision to Diff 448733. ychen added a comment. - update test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129954/new/ https://reviews.llvm.org/D129954 Files: clang/lib/CodeGen/CGStmt.cpp clang/test/CodeGen/inline-asm-x86-f

[PATCH] D129954: [CodeGen][inlineasm] assume the flag output of inline asm is boolean value

ychen updated this revision to Diff 448732. ychen added a comment. - use < 2 - check multiple flag outputs Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129954/new/ https://reviews.llvm.org/D129954 Files: clang/lib/CodeGen/CGStmt.cpp clang/tes

[PATCH] D130710: [ASTWriter] Provide capability to output a PCM/PCH file that does not write out information about its output path

This revision was automatically updated to reflect the committed changes. Closed by commit rG944a86de7c50: [ASTWriter] Provide capability to output a PCM/PCH file that does not write out… (authored by akyrtzi). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llv

[clang] 944a86d - [ASTWriter] Provide capability to output a PCM/PCH file that does not write out information about its output path

Author: Argyrios Kyrtzidis Date: 2022-07-29T15:21:54-07:00 New Revision: 944a86de7c504121da8958b37b7fe8fff53d43a2 URL: https://github.com/llvm/llvm-project/commit/944a86de7c504121da8958b37b7fe8fff53d43a2 DIFF: https://github.com/llvm/llvm-project/commit/944a86de7c504121da8958b37b7fe8fff53d43a2.

[clang] 65c9265 - [clang-repl] Disable exectuion unitests on unsupported platform by lljit instance test.

Author: Sunho Kim Date: 2022-07-30T07:18:04+09:00 New Revision: 65c9265f4158512e8dc85558ef8e73c0714e90c3 URL: https://github.com/llvm/llvm-project/commit/65c9265f4158512e8dc85558ef8e73c0714e90c3 DIFF: https://github.com/llvm/llvm-project/commit/65c9265f4158512e8dc85558ef8e73c0714e90c3.diff LOG

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

sunho added a comment. In D130786#3688387 , @alanphipps wrote: > I'm still seeing a failure on my downstream Arm compiler on Linux in the unit > tests -- I thought I saw the same failure on the buildbots: > > FAIL: llvm_regressions :: > Clang-Unit/In

[PATCH] D130801: [clang-tidy] Add trace code for ASTVisitors

njames93 created this revision. njames93 added reviewers: alexfh, aaron.ballman. Herald added subscribers: carlosgalvezp, xazax.hun, mgorny. Herald added a project: All. njames93 requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commits. T

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

alanphipps added a comment. I'm still seeing a failure on my downstream Arm compiler on Linux in the unit tests -- I thought I saw the same failure on the buildbots: FAIL: llvm_regressions :: Clang-Unit/Interpreter/ClangReplInterpreterTests/1/8 3

[PATCH] D122768: [Clang][C++20] Support capturing structured bindings in lambdas

cor3ntin updated this revision to Diff 448726. cor3ntin added a comment. Fix codegen test (Thanks Erich!) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D122768/new/ https://reviews.llvm.org/D122768 Files: clang-tools-extra/clang-tidy/modernize/Lo

[PATCH] D129973: [clang] Pass FoundDecl to DeclRefExpr creator for operator overloads

yvvan added a comment. In D129973#3688329 , @SimplyDanny wrote: > In D129973#3684940 , @yvvan wrote: > >> With this change we don't pass "LocInfo" directly and it seems to break the >> locations when calling "ge

[PATCH] D130800: [clang][Headers] Avoid compiler warnings in builtin headers

ddcc created this revision. ddcc added reviewers: efriedma, rnk, aaron.ballman. Herald added a project: All. ddcc requested review of this revision. Herald added a project: clang. While debugging module support using -Wsystem-headers, we discovered that if -Werror, and -Wundef or -Wmacro-redefined

[PATCH] D130710: [ASTWriter] Provide capability to output a PCM/PCH file that does not write out information about its output path

akyrtzi added a comment. In D130710#3685436 , @benlangmuir wrote: > Is the functionality provided by `ORIGINAL_PCH_DIR` still useful for making > it possible to move a PCH and its referenced headers? It's not completely > clear to me when this feature

[PATCH] D130710: [ASTWriter] Provide capability to output a PCM/PCH file that does not write out information about its output path

akyrtzi updated this revision to Diff 448715. akyrtzi added a comment. Add `FIXME` comment to consider either removing `ORIGINAL_PCH_DIR` or changing the default. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130710/new/ https://reviews.llvm.org/D

[PATCH] D130577: clang-driver: use llvm_fast_shutdown

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

[PATCH] D129973: [clang] Pass FoundDecl to DeclRefExpr creator for operator overloads

SimplyDanny added a comment. In D129973#3684940 , @yvvan wrote: > With this change we don't pass "LocInfo" directly and it seems to break the > locations when calling "getCXXOperatorNameRange" for this DeclRefExpr later > on. Please fix it. You can intr

[PATCH] D130799: [clang] Keep declaration name info in DeclRefExpr

SimplyDanny created this revision. SimplyDanny added a reviewer: yvvan. Herald added a subscriber: arphaman. Herald added a project: All. SimplyDanny requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Without it `getCXXOperatorNameRange` retur

[PATCH] D129118: CommandLine: add and use cl::SubCommand::get{All,TopLevel}

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

[PATCH] D128490: [ODRHash diagnostics] Transform method `ASTReader::diagnoseOdrViolations` into a class `ODRDiagsEmitter`. NFC.

vsapsai marked 2 inline comments as done. vsapsai added inline comments. Comment at: clang/lib/Serialization/ASTReader.cpp:9193-9195 - // Otherwise, use the name of the top-level module the decl is within. - if (ModuleFile *M = getOwningModuleFile(D)) -return M->ModuleName;

[PATCH] D128490: [ODRHash diagnostics] Transform method `ASTReader::diagnoseOdrViolations` into a class `ODRDiagsEmitter`. NFC.

vsapsai updated this revision to Diff 448710. vsapsai added a comment. - Address review comments. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D128490/new/ https://reviews.llvm.org/D128490 Files: clang/include/clang/AST/DeclCXX.h clang/include

[PATCH] D126676: [clang] Disallow differences in defines used for creating and using PCH

dexonsmith added a comment. In D126676#3687491 , @mstorsjo wrote: > In D126676#3687227 , @dexonsmith > wrote: > >> There are cases where it’s safe to have mismatched defines (e.g., if a >> define can be proven d

[PATCH] D130516: [llvm] compression classes

dblaikie added a comment. In D130516#3688123 , @MaskRay wrote: > I'd like to make a few arguments for the current namespace+free function > design, as opposed to the class+member function design as explored in this > patch (but thanks for the exploratio

[PATCH] D126676: [clang] Disallow differences in defines used for creating and using PCH

mstorsjo updated this revision to Diff 448708. mstorsjo added a comment. Update the parameters to the internal `checkPreprocessorOptions` function to use a three-state enum instead of having two separate `bool` flags. The 15.x release already has been branched, but I'd like to have this backpor

[PATCH] D129798: [clang-tidy] Rename the make-confusable-table executable

mstorsjo added inline comments. Comment at: llvm/utils/gn/secondary/clang-tools-extra/clang-tidy/misc/ConfusableTable/BUILD.gn:1 -executable("make-confusable-table") { +executable("clang-tidy-confusable-chars-gen") { deps = [ "//llvm/lib/Support" ] thakis wro

[PATCH] D130726: [clang][dataflow] Handle multiple context-sensitive calls to the same function

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGa6ddc6848782: [clang][dataflow] Handle multiple context-sensitive calls to the same function (authored by samestep). Changed prior to commit: http

[clang] a6ddc68 - [clang][dataflow] Handle multiple context-sensitive calls to the same function

Author: Sam Estep Date: 2022-07-29T19:40:19Z New Revision: a6ddc68487823d48c0ec0ddd649ace4a2732d0b0 URL: https://github.com/llvm/llvm-project/commit/a6ddc68487823d48c0ec0ddd649ace4a2732d0b0 DIFF: https://github.com/llvm/llvm-project/commit/a6ddc68487823d48c0ec0ddd649ace4a2732d0b0.diff LOG: [cl

[PATCH] D128490: [ODRHash diagnostics] Transform method `ASTReader::diagnoseOdrViolations` into a class `ODRDiagsEmitter`. NFC.

vsapsai planned changes to this revision. vsapsai added inline comments. Comment at: clang/lib/Serialization/ASTReader.cpp:9577-9584 +static std::string getOwningModuleNameForDiagnostic(const Decl *D) { + // If we know the owning module, use it. + if (Module *M = D->getImported

[PATCH] D130726: [clang][dataflow] Handle multiple context-sensitive calls to the same function

ymandel accepted this revision. ymandel added a comment. Nice work! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130726/new/ https://reviews.llvm.org/D130726 ___ cfe-commits mailing list cfe-commits@lis

[PATCH] D130058: [Clang] Diagnose ill-formed constant expression when setting a non fixed enum to a value outside the range of the enumeration values

shafik added a comment. In D130058#3688049 , @thakis wrote: > Here's a reduced repro of one case: > > % cat test.cc > typedef enum VkResult { > VK_ERROR_INVALID_OPAQUE_CAPTURE_ADDRESS = -1000257000, > VK_RESULT_MAX_ENUM = 0x7FFF > }

[PATCH] D130794: [Docs] Add HLSL ResourceType documentation

beanz created this revision. beanz added reviewers: aaron.ballman, pow2clk, python3kgae, bogner, MaskRay. Herald added subscribers: Anastasia, StephenFan, arphaman. Herald added a project: All. beanz requested review of this revision. Herald added a project: clang. Along with the new documentation

[PATCH] D130726: [clang][dataflow] Handle multiple context-sensitive calls to the same function

samestep updated this revision to Diff 448698. samestep added a comment. Add more comments and tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130726/new/ https://reviews.llvm.org/D130726 Files: clang/include/clang/Analysis/FlowSensitive/Dat

[clang] 09c4121 - Revert "Revert "[Clang][Attribute] Introduce maybe_undef attribute for function arguments which accepts undef values""

Author: skc7 Date: 2022-07-29T19:07:07Z New Revision: 09c41211231578765cdb296bda3b4c7f3c22b4ba URL: https://github.com/llvm/llvm-project/commit/09c41211231578765cdb296bda3b4c7f3c22b4ba DIFF: https://github.com/llvm/llvm-project/commit/09c41211231578765cdb296bda3b4c7f3c22b4ba.diff LOG: Revert "

[PATCH] D130516: [llvm] compression classes

leonardchan added inline comments. Comment at: llvm/lib/Support/Compression.cpp:30-32 +ZStdCompressionAlgorithm +*llvm::compression::ZStdCompressionAlgorithm::Instance = +new ZStdCompressionAlgorithm(); Perhaps for each of these, you could instead hav

[PATCH] D130516: [llvm] compression classes

MaskRay added a comment. I'd like to make a few arguments for the current namespace+free function design, as opposed to the class+member function design as explored in this patch (but thanks for the exploration!). Let's discuss several use cases. (a) if a use case just calls compress/uncompress

[PATCH] D129131: Remove uses of llvm_shutdown

Meinersbur added inline comments. Comment at: polly/lib/External/isl/interface/extract_interface.cc:590 delete Clang; - llvm::llvm_shutdown(); This file is imported from the upstream project (https://repo.or.cz/isl.git/blob/295cf91923295ca694e9e4

[PATCH] D130058: [Clang] Diagnose ill-formed constant expression when setting a non fixed enum to a value outside the range of the enumeration values

erichkeane added a comment. In D130058#3688049 , @thakis wrote: > Here's a reduced repro of one case: > > % cat test.cc > typedef enum VkResult { > VK_ERROR_INVALID_OPAQUE_CAPTURE_ADDRESS = -1000257000, > VK_RESULT_MAX_ENUM = 0x7FFF >

[PATCH] D130793: [clang-tidy] adjust treating of array-of-pointers when 'AnalyzePointers' is deactivated

JonasToth created this revision. JonasToth added reviewers: njames93, aaron.ballman, alexfh. Herald added subscribers: carlosgalvezp, xazax.hun. Herald added a project: All. JonasToth requested review of this revision. Herald added a project: clang-tools-extra. Herald added a subscriber: cfe-commit

[PATCH] D130363: [clang] Give priority to Class context while parsing declarations

furkanusta added a comment. I've added a test case but I have a question. This is regardless of the current issue (i.e. no function context, clang++14 with no patches) struct X { virtual void foo(); }; struct Y : public X { over }; I am trying to complete override in class Y here, b

[PATCH] D130363: [clang] Give priority to Class context while parsing declarations

furkanusta updated this revision to Diff 448694. furkanusta added a comment. - [clang] Add test case for D130363 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130363/new/ https://reviews.llvm.org/D130363 Files:

[PATCH] D130791: [clang] Short-circuit trivial constexpr array constructors

tbaeder created this revision. tbaeder added reviewers: aaron.ballman, rsmith. Herald added a project: All. tbaeder requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. As discussed in https://github.com/llvm/llvm-project/issues/56774 Reposito

[PATCH] D130224: [Clang][Attribute] Introduce maybe_undef attribute for function arguments which accepts undef values

amyk added a comment. In D130224#3688034 , @skc7 wrote: > In D130224#3687907 , @aaron.ballman > wrote: > >> In D130224#3687860 , @amyk wrote: >> >>> In D130224#3687487

[PATCH] D130224: [Clang][Attribute] Introduce maybe_undef attribute for function arguments which accepts undef values

amyk added a comment. In D130224#3687907 , @aaron.ballman wrote: > In D130224#3687860 , @amyk wrote: > >> In D130224#3687487 , @mubarizafzal >> wrote: >> >>> Hi, the tes

[PATCH] D130058: [Clang] Diagnose ill-formed constant expression when setting a non fixed enum to a value outside the range of the enumeration values

thakis added a comment. Here's a reduced repro of one case: % cat test.cc typedef enum VkResult { VK_ERROR_INVALID_OPAQUE_CAPTURE_ADDRESS = -1000257000, VK_RESULT_MAX_ENUM = 0x7FFF } VkResult; constexpr VkResult VK_FAKE_DEVICE_OOM_FOR_TESTING = static_cast(VK_RESULT_MA

[clang] 4e1fe96 - Revert "[Clang][Attribute] Introduce maybe_undef attribute for function arguments which accepts undef values"

Author: Amy Kwan Date: 2022-07-29T13:28:48-05:00 New Revision: 4e1fe968c9de73507a1bf0c8aa57e06be457816e URL: https://github.com/llvm/llvm-project/commit/4e1fe968c9de73507a1bf0c8aa57e06be457816e DIFF: https://github.com/llvm/llvm-project/commit/4e1fe968c9de73507a1bf0c8aa57e06be457816e.diff LOG:

[PATCH] D130224: [Clang][Attribute] Introduce maybe_undef attribute for function arguments which accepts undef values

skc7 added a comment. In D130224#3687907 , @aaron.ballman wrote: > In D130224#3687860 , @amyk wrote: > >> In D130224#3687487 , @mubarizafzal >> wrote: >> >>> Hi, the tes

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

sunho added a comment. In D130786#3688019 , @probinson wrote: > If you're going to post a patch for review, you really should wait for > someone to review it before you land it. I rushed as bots were breaking on the upstream. Sorry about that. Reposit

[PATCH] D130790: Fix failing tests for "[Clang][Attribute] Introduce maybe_undef attribute for function arguments which accepts undef values"

skc7 created this revision. skc7 added reviewers: aaron.ballman, ronlieb, arsenm. Herald added a subscriber: pengfei. Herald added a project: All. skc7 requested review of this revision. Herald added subscribers: cfe-commits, wdng. Herald added a project: clang. Add target triple x86_64-gnu-linux

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

probinson added a comment. If you're going to post a patch for review, you really should wait for someone to review it before you land it. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130786/new/ https://reviews.llvm.org/D130786

[PATCH] D93138: Add initial support for multilibs in Baremetal toolchain.

abidh added inline comments. Comment at: clang/lib/Driver/ToolChains/BareMetal.cpp:177 if (!getDriver().SysRoot.empty()) -return getDriver().SysRoot; +return getDriver().SysRoot + SelectedMultilib.osSuffix(); zixuan-wu wrote: > I think the multilib o

[PATCH] D124750: [MLIR] Add a utility to sort the operands of commutative ops

srishti-pm updated this revision to Diff 448682. srishti-pm marked 3 inline comments as done. srishti-pm added a comment. Addressed the final NITs. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D124750/new/ https://reviews.llvm.org/D124750 Files:

[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

sunho added a comment. https://reviews.llvm.org/D130788 this is the patch fyi. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129242/new/ https://reviews.llvm.org/D129242 ___ cfe-commits mailing list cfe-

[PATCH] D129242: [clang-repl] Add host exception support check utility flag.

sunho added a comment. @uabelho The thing is we have to statically link libstdc++ in JIT stack, but this is not possible at the moment in many platforms. It's actually possible with a new generation linker called JITLink, but it has limited platform support. For now, I just sumbitted a patch di

[PATCH] D124750: [MLIR] Add a utility to sort the operands of commutative ops

Mogball added a comment. I'm referring to the nitty gritty details of the while loop inside the comparator. It looks pretty tight to me right now. If the operands are already sorted, worst-case each operand is compared against only its neighbours. Unfortunately, without extra bookkeeping, BFS w

[PATCH] D124750: [MLIR] Add a utility to sort the operands of commutative ops

mehdi_amini added a comment. > . I haven't thought too hard about the performance of that while loop but it > seems good enough to land for now. What's the finality of it? That is: outside of canonicalization what is its purpose? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTIO

[PATCH] D130224: [Clang][Attribute] Introduce maybe_undef attribute for function arguments which accepts undef values

aaron.ballman added a comment. In D130224#3687860 , @amyk wrote: > In D130224#3687487 , @mubarizafzal > wrote: > >> Hi, the test cases that this patch introduces are failing on some ppc64le >> (Linux on Power) b

[PATCH] D130786: [clang-repl] Disable execution unittests on unsupported platforms.

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 rG4191d661c746: [clang-repl] Disable execution unittests on unsuppor

[clang] 4191d66 - [clang-repl] Disable execution unittests on unsupported platforms.

Author: Sunho Kim Date: 2022-07-30T02:28:03+09:00 New Revision: 4191d661c74622c6fa72c1643e4567f45e6c9e1b URL: https://github.com/llvm/llvm-project/commit/4191d661c74622c6fa72c1643e4567f45e6c9e1b DIFF: https://github.com/llvm/llvm-project/commit/4191d661c74622c6fa72c1643e4567f45e6c9e1b.diff LOG

[PATCH] D124750: [MLIR] Add a utility to sort the operands of commutative ops

Mogball accepted this revision. Mogball added a comment. This revision is now accepted and ready to land. Got a few small nits but otherwise LGTM. Thanks for all the hard work! This looks really solid now. I haven't thought too hard about the performance of that while loop but it seems good enou

[clang] d8352ab - Diagnose use of _Noreturn on a struct/union field

Author: Aaron Ballman Date: 2022-07-29T13:18:44-04:00 New Revision: d8352abd3a4f411828dbe46c7dfd3e935ab4dd4a URL: https://github.com/llvm/llvm-project/commit/d8352abd3a4f411828dbe46c7dfd3e935ab4dd4a DIFF: https://github.com/llvm/llvm-project/commit/d8352abd3a4f411828dbe46c7dfd3e935ab4dd4a.diff

[PATCH] D130058: [Clang] Diagnose ill-formed constant expression when setting a non fixed enum to a value outside the range of the enumeration values

aaron.ballman added a comment. In D130058#3687680 , @thakis wrote: > Hello, this breaks a bunch of existing code over here (and I imagine > elsewhere). Do you have an idea on how much a bunch is and whether the fixes are particularly involved or not? D

[PATCH] D130224: [Clang][Attribute] Introduce maybe_undef attribute for function arguments which accepts undef values

amyk added a comment. In D130224#3687487 , @mubarizafzal wrote: > Hi, the test cases that this patch introduces are failing on some ppc64le > (Linux on Power) buildbots: > https://lab.llvm.org/buildbot/#/builders/57 > https://lab.llvm.org/buildbot/#/bui

[PATCH] D54943: [clang-tidy] implement new check 'misc-const-correctness' to add 'const' to unmodified variables

JonasToth added a comment. In D54943#3681527 , @sammccall wrote: > This check is enabled by default in LLVM (`Checks: misc-*` in > `llvm-project/.clang-tidy`) > > The warning on mutable non-ref local variables is pretty noisy: a *lot* of > existing code

[PATCH] D130726: [clang][dataflow] Handle multiple context-sensitive calls to the same function

xazax.hun accepted this revision. xazax.hun added a comment. This revision is now accepted and ready to land. I'd love to see some comments in the source code to make sure one will not sue these facilities for other purposes that this was not designed to do. Alternatively, `ContextSensitive` opt

[PATCH] D127403: [clangd] Implement semantic token modifier "definition"

ckandeler updated this revision to Diff 448655. ckandeler added a comment. Addressed remaining Objective-C issues Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D127403/new/ https://reviews.llvm.org/D127403 Files: clang-tools-extra/clangd/Semantic

[PATCH] D130058: [Clang] Diagnose ill-formed constant expression when setting a non fixed enum to a value outside the range of the enumeration values

tahonermann added inline comments. Comment at: clang/docs/ReleaseNotes.rst:55-57 +- Clang will now correctly diagnose as ill-formed a constant expression where an + enum without a fixed underlying type is set to a value outside the range of + of the enumerations values. Fixes

[PATCH] D130058: [Clang] Diagnose ill-formed constant expression when setting a non fixed enum to a value outside the range of the enumeration values

thakis added a comment. Hello, this breaks a bunch of existing code over here (and I imagine elsewhere). Can we make this a warning that's mapped to an error by default, so that people can turn it off for a while while they fix their code? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D

[PATCH] D130750: [Clang] Do not check for underscores in isAllowedInitiallyIDChar

cor3ntin added a comment. Thank you both! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130750/new/ https://reviews.llvm.org/D130750 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.

[PATCH] D130735: [Clang][LLD][cmake] Drop deprecated support for `llvm-config`-based build

compnerd accepted this revision. compnerd added a comment. This revision is now accepted and ready to land. Since it has been deprecated for 2 releases, it seems reasonable to cleanup. This seems good to me from the build side, please do wait a bit for anyone else to chime in on this. Reposit

[PATCH] D130750: [Clang] Do not check for underscores in isAllowedInitiallyIDChar

This revision was automatically updated to reflect the committed changes. Closed by commit rGad16268f1350: [Clang] Do not check for underscores in isAllowedInitiallyIDChar (authored by cor3ntin). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130750/n

[clang] ad16268 - [Clang] Do not check for underscores in isAllowedInitiallyIDChar

Author: Corentin Jabot Date: 2022-07-29T17:46:38+02:00 New Revision: ad16268f135001bd21a805ae8acf8d8243793984 URL: https://github.com/llvm/llvm-project/commit/ad16268f135001bd21a805ae8acf8d8243793984 DIFF: https://github.com/llvm/llvm-project/commit/ad16268f135001bd21a805ae8acf8d8243793984.diff

[PATCH] D130750: [Clang] Do not check for underscores in isAllowedInitiallyIDChar

tahonermann accepted this revision. tahonermann added a comment. LGTM, thanks Corentin. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130750/new/ https://reviews.llvm.org/D130750 ___ cfe-commits mailing

[PATCH] D122768: [Clang][C++20] Support capturing structured bindings in lambdas

erichkeane added inline comments. Comment at: clang/test/CodeGenCXX/cxx20-decomposition.cpp:26 +// CHECK: %{{.*}} = load ptr, {{.*}} +// CHECK: %{{.*}} = load i32, {{.*}} +// CHECK: %{{.*}} = getelementptr {{.*}}, i32 0, i32 0 cor3ntin wrote: > erichkeane wrote:

[PATCH] D122768: [Clang][C++20] Support capturing structured bindings in lambdas

cor3ntin added inline comments. Comment at: clang/lib/Sema/SemaExpr.cpp:18399 + if (Diagnose) +diagnoseUncapturableValueReferenceOrBinding(S, Loc, Var); + return false; erichkeane wrote: > first, why are we checking for !CPlusPlus... aren't Bind

[PATCH] D129231: [Builtins] Do not claim all libfuncs are readnone with trapping math.

fhahn updated this revision to Diff 448628. fhahn added a comment. In D129231#3638946 , @john.brawn wrote: > Looking at the descriptions of maths functions in C99 (and I expect C11 will > be the same) it looks like there are three kinds: > > - Those t

[PATCH] D127812: [AArch64] Function multiversioning support added.

danielkiss requested changes to this revision. danielkiss added a comment. This revision now requires changes to proceed. Please find the thread on the GCC mailing list here: https://gcc.gnu.org/pipermail/gcc/2022-July/239134.html Feedback there sounds positive to me. All received feedback added

[PATCH] D130777: Enable embedded lto for XCOFF.

sfertile created this revision. sfertile added reviewers: hubert.reinterpretcast, DiggerLin, nemanjai, MaskRay, tejohnson, mehdi_amini, phosek, arda. Herald added subscribers: ormris, StephenFan, steven_wu, kbarton, hiraditya, inglorion, mgorny. Herald added a project: All. sfertile requested rev

[PATCH] D126676: [clang] Disallow differences in defines used for creating and using PCH

mstorsjo added a comment. In D126676#3687227 , @dexonsmith wrote: > I haven’t reviewed the details of the patch and won’t have time to do so (at > least for a while), but the description of the intended (more narrow) scope > SGTM. With the scope limite

  1   2   >