[PATCH] D121426: [VFS] Update uses of getVFSFromYAML to use the new getVFSFromYAMLs API

bnbarham marked an inline comment as done. bnbarham added inline comments. Comment at: llvm/include/llvm/Support/Error.h:1284 - StringRef getFileName() { return FileName; } + StringRef getFileName() const { return FileName; } dexonsmith wrote: > bnbarham

[PATCH] D121426: [VFS] Update uses of getVFSFromYAML to use the new getVFSFromYAMLs API

bnbarham added a comment. In D121426#3376442 , @dexonsmith wrote: >> Includes two test fixes (since chained mappings are no longer allowed) >> and adds a new test for multiple overlays. > > Are we sure no one needs to support chained mappings? Has there

[PATCH] D118948: [MTE] Add -fsanitize=memtag* and friends.

MaskRay added inline comments. Comment at: clang/include/clang/Driver/SanitizerArgs.h:67 + std::string MemtagMode = ""; + Comment at: clang/lib/Driver/SanitizerArgs.cpp:658 +

[PATCH] D120639: [RISCV] Pass -mno-relax to assembler when -fno-integrated-as specified

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

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

pengfei accepted this revision. pengfei added a comment. LGTM. Comment at: clang/lib/Basic/Targets/X86.cpp:1133 +#include "llvm/Support/X86TargetParser.def" +.Default(""); +} clang-format. Comment at:

[PATCH] D121177: [modules] Merge equivalent extensions and diagnose ivar redeclarations for extensions loaded from different modules.

vsapsai marked 6 inline comments as done. vsapsai added a comment. In D121177#3371832 , @ChuanqiXu wrote: > I see it is guaranteed by `lookupInstanceVariable(Identifier)` now. I don't > know much about ObjC. But I feel a little bit strange. Is it

[PATCH] D120296: [Attr] Fix a btf_type_tag AST generation bug

yonghong-song added inline comments. Comment at: clang/include/clang/AST/TypeLoc.h:923 + void initializeLocal(ASTContext , SourceLocation loc) {} + + QualType getInnerType() const { return getTypePtr()->getWrappedType(); } aaron.ballman wrote: > Do we also

[PATCH] D121176: [ASTStructuralEquivalence] Add support for comparing ObjCCategoryDecl.

vsapsai added a comment. In D121176#3372193 , @martong wrote: > Basically, this looks good to me. But my confidence with ObjC is low, so, I'd > like @shafik as well to approve. > Besides, seems like the pre-merge check fails with the new tests on

[PATCH] D121177: [modules] Merge equivalent extensions and diagnose ivar redeclarations for extensions loaded from different modules.

vsapsai updated this revision to Diff 414780. vsapsai added a comment. Address comments and make the tests use non-fragile ABI (it's not default everywhere). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121177/new/

[PATCH] D121426: [VFS] Update uses of getVFSFromYAML to use the new getVFSFromYAMLs API

dexonsmith added a comment. > Includes two test fixes (since chained mappings are no longer allowed) > and adds a new test for multiple overlays. Are we sure no one needs to support chained mappings? Has there been a ~~clang-dev~~ discourse discussion about it already? Just concerned that some

[PATCH] D121175: [clang] Add -Wstart-no-unknown-warning-option/-Wend-no-unknown-warning-option.

MaskRay added a comment. I am also unsure about the usefulness. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121175/new/ https://reviews.llvm.org/D121175 ___ cfe-commits mailing list

[PATCH] D121049: [Clang][VE] Add vector load intrinsics

This revision was automatically updated to reflect the committed changes. Closed by commit rGb1b4b6f36695: [Clang][VE] Add vector load intrinsics (authored by kaz7). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121049/new/

[clang] b1b4b6f - [Clang][VE] Add vector load intrinsics

Author: Kazushi (Jam) Marukawa Date: 2022-03-12T09:09:57+09:00 New Revision: b1b4b6f366956a5fd1bfccb3d7c2f7519ed45a28 URL: https://github.com/llvm/llvm-project/commit/b1b4b6f366956a5fd1bfccb3d7c2f7519ed45a28 DIFF:

[PATCH] D37624: add support for -fno-instrument-functions and -finstrument-functions-exclude-{file,function}-list= to match gcc options.

hiraditya added a comment. Herald added a project: All. Is there a plan to revive this? this is quite necessary for one of my use cases. can we at least have '-fno-instrument-functions'? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D37624/new/ https://reviews.llvm.org/D37624

[clang] 0c0f6cf - [CMake] Rename TARGET_TRIPLE to LLVM_TARGET_TRIPLE

Author: Petr Hosek Date: 2022-03-11T15:43:01-08:00 New Revision: 0c0f6cfb7b9fab0ee69ef67ed95c194698c9d794 URL: https://github.com/llvm/llvm-project/commit/0c0f6cfb7b9fab0ee69ef67ed95c194698c9d794 DIFF: https://github.com/llvm/llvm-project/commit/0c0f6cfb7b9fab0ee69ef67ed95c194698c9d794.diff

[clang-tools-extra] 0c0f6cf - [CMake] Rename TARGET_TRIPLE to LLVM_TARGET_TRIPLE

Author: Petr Hosek Date: 2022-03-11T15:43:01-08:00 New Revision: 0c0f6cfb7b9fab0ee69ef67ed95c194698c9d794 URL: https://github.com/llvm/llvm-project/commit/0c0f6cfb7b9fab0ee69ef67ed95c194698c9d794 DIFF: https://github.com/llvm/llvm-project/commit/0c0f6cfb7b9fab0ee69ef67ed95c194698c9d794.diff

[PATCH] D119918: [CMake] Rename TARGET_TRIPLE to LLVM_TARGET_TRIPLE

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0c0f6cfb7b9f: [CMake] Rename TARGET_TRIPLE to LLVM_TARGET_TRIPLE (authored by phosek). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D121498: let EST_Uninstantiated in FunctionProtoType::canThrow return CT_Dependent

zhouyizhou created this revision. zhouyizhou added reviewers: rsmith, erichkeane, CornedBee. Herald added a project: All. zhouyizhou requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. When compile following code without -std=c++17, clang will

[PATCH] D121497: Lex: add support for `{,u}i128` Microsoft extension

compnerd created this revision. compnerd added a reviewer: aaron.ballman. compnerd added a project: clang. Herald added a project: All. compnerd requested review of this revision. Herald added a subscriber: cfe-commits. The Windows SDK has occurrences of the `i128` and `ui128` suffix (at least in

[clang] 46626bc - [NFC] Improve comment and rename test file

Author: Julian Lettner Date: 2022-03-11T14:47:54-08:00 New Revision: 46626bc87382cad7a0c5ba497b3454337c45faa8 URL: https://github.com/llvm/llvm-project/commit/46626bc87382cad7a0c5ba497b3454337c45faa8 DIFF:

[PATCH] D121450: [clang-format] Handle attributes before case label.

owenpan accepted this revision. owenpan added inline comments. Comment at: clang/lib/Format/UnwrappedLineParser.cpp:578 +break; + LLVM_FALLTHROUGH; Can we remove this empty line? Comment at:

[PATCH] D121426: [VFS] Update uses of getVFSFromYAML to use the new getVFSFromYAMLs API

bnbarham updated this revision to Diff 414751. bnbarham edited the summary of this revision. bnbarham added a comment. Update to single review dependency Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121426/new/ https://reviews.llvm.org/D121426

[PATCH] D121434: [clang-format][NFC] Group QualifierAlignment with other C++ passes

This revision was automatically updated to reflect the committed changes. Closed by commit rG9f616a467fc7: [clang-format][NFC] Group QualifierAlignment with other C++ passes (authored by owenpan). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 9f616a4 - [clang-format][NFC] Group QualifierAlignment with other C++ passes

Author: owenca Date: 2022-03-11T14:18:00-08:00 New Revision: 9f616a467fc710f084b7e57812a2ed64c214c2c6 URL: https://github.com/llvm/llvm-project/commit/9f616a467fc710f084b7e57812a2ed64c214c2c6 DIFF: https://github.com/llvm/llvm-project/commit/9f616a467fc710f084b7e57812a2ed64c214c2c6.diff LOG:

[clang-tools-extra] 602ea53 - [clang-tidy] Refactor: Prefer single quoted strings over double quoted strings [NFC]

Author: Richard Date: 2022-03-11T15:08:24-07:00 New Revision: 602ea53cb7e7119f7ba33239ca8ead377f2fcbd4 URL: https://github.com/llvm/llvm-project/commit/602ea53cb7e7119f7ba33239ca8ead377f2fcbd4 DIFF: https://github.com/llvm/llvm-project/commit/602ea53cb7e7119f7ba33239ca8ead377f2fcbd4.diff LOG:

[PATCH] D109701: [clang] Emit SARIF Diagnostics: Create `clang::SarifDocumentWriter` interface

vaibhav.y added a comment. Hi, Apologies for the long delay! I was on a short break to focus to other projects. I have some changes in mind such as: - Creating the `SarifLog` object to represent the top-level document. Currently we store this as a JSON object which ends up rather mucky.

[PATCH] D109701: [clang] Emit SARIF Diagnostics: Create `clang::SarifDocumentWriter` interface

vaibhav.y updated this revision to Diff 414743. vaibhav.y marked 15 inline comments as done. vaibhav.y added a comment. Herald added a project: All. Fixup comments Mark clang::FullSourceRange as returning const & to its locs rebase on upstream main Repository: rG LLVM Github Monorepo CHANGES

[PATCH] D111400: [Clang] Implement P2242R3

hubert.reinterpretcast added a comment. In D111400#3376070 , @hubert.reinterpretcast wrote: > In C++2b, the `-Wc++20-compat` warning is produced for the places where we > warn above and is not produced for the places where we don't warn above. I'm not

[PATCH] D121122: Set FLT_EVAL_METHOD to -1 when fast-math is enabled.

zahiraam updated this revision to Diff 414739. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121122/new/ https://reviews.llvm.org/D121122 Files: clang/include/clang/Lex/Preprocessor.h clang/lib/Lex/PPMacroExpansion.cpp clang/lib/Sema/Sema.cpp clang/lib/Sema/SemaAttr.cpp

[PATCH] D111400: [Clang] Implement P2242R3

hubert.reinterpretcast added a comment. @aaron.ballman, I haven't had a chance to look at the code in detail yet, but I would like to get on the same page about the error/warning behaviour. In the older modes, we stay conforming either - by producing the warning (when we know the user said

[PATCH] D121122: Set FLT_EVAL_METHOD to -1 when fast-math is enabled.

zahiraam added inline comments. Comment at: clang/test/CodeGen/eval-method-fast-math.c:113-116 +#pragma float_control(precise, on) +int val2 = __FLT_EVAL_METHOD__; + } + return __FLT_EVAL_METHOD__; aaron.ballman wrote: > Shouldn't there be CHECK lines for

[PATCH] D121076: [MLIR][Python] Add SCFIfOp Python binding

mehdi_amini added a comment. Can you rebase? Your patch does not apply apparently Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121076/new/ https://reviews.llvm.org/D121076 ___ cfe-commits mailing list

[PATCH] D120610: [DebugInfo] Include DW_TAG_skeleton_unit when looking for parent UnitDie

wenlei added a comment. In D120610#3376002 , @dblaikie wrote: > It'd be good to include some testing beyond "does not crash" - like what was > the specific debug info we were trying to create when the crash was hit? > Perhaps we should be testing that

[PATCH] D120610: [DebugInfo] Include DW_TAG_skeleton_unit when looking for parent UnitDie

wenlei updated this revision to Diff 414737. wenlei added a comment. Add test check for template type parameters Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120610/new/ https://reviews.llvm.org/D120610 Files:

[PATCH] D121122: Set FLT_EVAL_METHOD to -1 when fast-math is enabled.

aaron.ballman added a comment. It looks like there are some clang-format issues to handle as well. Comment at: clang/test/CodeGen/eval-method-fast-math.c:113-116 +#pragma float_control(precise, on) +int val2 = __FLT_EVAL_METHOD__; + } + return __FLT_EVAL_METHOD__;

[PATCH] D117522: [clang-tidy] Add modernize-macro-to-enum check

aaron.ballman added a comment. In D117522#3364387 , @LegalizeAdulthood wrote: > Ping. Another week waiting for reviews... Thanks for the ping! FWIW, it's also not uncommon for there to be week delays (reviewers go on vacation, have other obligations,

[PATCH] D120610: [DebugInfo] Include DW_TAG_skeleton_unit when looking for parent UnitDie

dblaikie added a comment. It'd be good to include some testing beyond "does not crash" - like what was the specific debug info we were trying to create when the crash was hit? Perhaps we should be testing that (since the crash demonstrates we weren't testing that anywhere else) Repository:

[PATCH] D121214: [clang-tidy][docs][NFC] Refer to the CERT rule in bugprone-shared-ptr-array-mismatch docs

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, I think this is a good incremental improvement until the check covers more of MEM51-CPP. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121214/new/

[PATCH] D121076: [MLIR][Python] Add SCFIfOp Python binding

chhzh123 added a comment. @ftynse Thanks for reviewing! Could you help land the PR? Seems I do not have access rights to push to the main repository. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121076/new/ https://reviews.llvm.org/D121076

[PATCH] D121122: Set FLT_EVAL_METHOD to -1 when fast-math is enabled.

zahiraam updated this revision to Diff 414729. zahiraam marked an inline comment as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121122/new/ https://reviews.llvm.org/D121122 Files: clang/include/clang/Lex/Preprocessor.h clang/lib/Lex/PPMacroExpansion.cpp

[PATCH] D121372: [clang-tidy][docs][NFC] Update URL and docs of PostfixOperatorCheck

aaron.ballman accepted this revision. aaron.ballman 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/D121372/new/ https://reviews.llvm.org/D121372

[PATCH] D120465: [clang][deps] Disable implicit module maps

dexonsmith accepted this revision. dexonsmith added a comment. This revision is now accepted and ready to land. LGTM, with a suggested change for the text of the comment. Comment at: clang/lib/Tooling/DependencyScanning/ModuleDepCollector.cpp:269-273 + // However, some module

[PATCH] D121176: [ASTStructuralEquivalence] Add support for comparing ObjCCategoryDecl.

vsapsai updated this revision to Diff 414724. vsapsai added a comment. Fix clang-format error. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121176/new/ https://reviews.llvm.org/D121176 Files: clang/include/clang/Testing/CommandLineArgs.h

[PATCH] D121373: [clang-tidy][docs] Fix wrong url in DontModifyStdNamespaceCheck

aaron.ballman accepted this revision. aaron.ballman 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/D121373/new/ https://reviews.llvm.org/D121373

[PATCH] D121426: [VFS] Update uses of getVFSFromYAML to use the new getVFSFromYAMLs API

bnbarham updated this revision to Diff 414718. bnbarham added a comment. Handle empty overlay file in clang tidy Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121426/new/ https://reviews.llvm.org/D121426 Files:

[PATCH] D121283: [Clang] Support multiple attributes in a single pragma

egorzhdan added a comment. In D121283#3373560 , @aaron.ballman wrote: > why do we support multiple attribute *specifiers* in the same pragma? I would > not expect to be able to mix attribute styles in the same pragma given that > all of the individual

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

aaron.ballman accepted this revision. aaron.ballman added a comment. LGTM, though I'm not qualified to review the CPU specific bits in the .def file. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121410/new/ https://reviews.llvm.org/D121410

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

erichkeane marked 4 inline comments as done. erichkeane added inline comments. Comment at: llvm/include/llvm/Support/X86TargetParser.def:236 +CPU_SPECIFIC("core_i7_sse4_2", "nehalem", 'P', "+cmov,+mmx,+sse,+sse2,+sse3,+ssse3,+sse4.1,+sse4.2,+popcnt")

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

erichkeane updated this revision to Diff 414709. erichkeane added a comment. Update the `core_aes_pclmulqdq` to be `westmere` CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121410/new/ https://reviews.llvm.org/D121410 Files: clang/include/clang/Basic/TargetInfo.h

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

craig.topper added inline comments. Comment at: llvm/include/llvm/Support/X86TargetParser.def:236 +CPU_SPECIFIC("core_i7_sse4_2", "nehalem", 'P', "+cmov,+mmx,+sse,+sse2,+sse3,+ssse3,+sse4.1,+sse4.2,+popcnt") +CPU_SPECIFIC("core_aes_pclmulqdq", "icelake-client", 'Q',

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

andrew.w.kaylor accepted this revision. andrew.w.kaylor added a comment. This revision is now accepted and ready to land. This looks good to me. Thanks for the patch! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121410/new/ https://reviews.llvm.org/D121410

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

erichkeane updated this revision to Diff 414699. erichkeane added a comment. Corrected the last few processor names thanks to @andrew.w.kaylor and @pengfei CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121410/new/ https://reviews.llvm.org/D121410 Files:

[PATCH] D88905: [Clang] Allow "ext_vector_type" applied to Booleans

aaron.ballman added a comment. In D88905#3362347 , @kaz7 wrote: > At the beginning, this implementation extends `vector_type` attribute which > is GCC's attribute. So, this may cause future conflicts with GCC when they > extend it. But, now this patch

[PATCH] D121468: [OpenMP] Add linking of OpenMP math library

jhuber6 added inline comments. Comment at: clang/include/clang/Driver/Options.td:1056 +def libomptarget_nvptx_math_bc_path_EQ : Joined<["--"], "libomptarget-nvptx-math-bc-path=">, + Group, HelpText<"Path to libomptarget-nvptx math bitcode library">; def dD : Flag<["-"],

[PATCH] D121468: [OpenMP] Add linking of OpenMP math library

JonChesterfield added inline comments. Comment at: clang/include/clang/Driver/Options.td:1056 +def libomptarget_nvptx_math_bc_path_EQ : Joined<["--"], "libomptarget-nvptx-math-bc-path=">, + Group, HelpText<"Path to libomptarget-nvptx math bitcode library">; def dD :

[PATCH] D121468: [OpenMP] Add linking of OpenMP math library

jdoerfert added inline comments. Comment at: clang/include/clang/Driver/Options.td:1056 +def libomptarget_nvptx_math_bc_path_EQ : Joined<["--"], "libomptarget-nvptx-math-bc-path=">, + Group, HelpText<"Path to libomptarget-nvptx math bitcode library">; def dD : Flag<["-"],

[PATCH] D121466: [OpenMP] Replace math headers with OpenMP wrapper calls

jdoerfert added a comment. We want one copy of this, yes. s/omp/llvm_gpu/, sure, though gpu might also not the best name. Generators, macros, etc. I'm not sold. Even if we want to do this, let's not conflate that. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D120187: [clang-tidy] Allow newline characters as separators for checks in Clang-Tidy configurations

aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM, but please wait a day or two before landing in case @njames93 has further concerns. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D121475: [Clang][Sema] Avoid crashing for `__builtin_memcpy_inline` with an array argument

egorzhdan created this revision. egorzhdan added a reviewer: gchatelet. Herald added a project: All. egorzhdan requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. This change teaches the Sema logic for `__builtin_memcpy_inline` to implicitly

[PATCH] D120296: [Attr] Fix a btf_type_tag AST generation bug

aaron.ballman added inline comments. Comment at: clang/include/clang/AST/TypeLoc.h:923 + void initializeLocal(ASTContext , SourceLocation loc) {} + + QualType getInnerType() const { return getTypePtr()->getWrappedType(); } Do we also need something like this?

[PATCH] D121135: [NFC][Clang][OpaquePtr] Remove calls to Address::deprecated in a couple more files

This revision was automatically updated to reflect the committed changes. Closed by commit rGaa4ea0ee5441: [NFC][Clang][OpaquePtr] Remove calls to Address::deprecated in a couple (authored by ahatanak). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] aa4ea0e - [NFC][Clang][OpaquePtr] Remove calls to Address::deprecated in a couple

Author: Akira Hatanaka Date: 2022-03-11T09:30:31-08:00 New Revision: aa4ea0ee54411d12d17dd6261f64726913f15e16 URL: https://github.com/llvm/llvm-project/commit/aa4ea0ee54411d12d17dd6261f64726913f15e16 DIFF:

[PATCH] D121455: [clang][dataflow] Add support for nested composite bool expressions

xazax.hun added inline comments. Comment at: clang/lib/Analysis/FlowSensitive/Transfer.cpp:516 +// assigned to it. +Visit(); +if (auto *Val = dyn_cast_or_null( Could you elaborate on when would this happen? I'd expect the traversal to always visit

[PATCH] D121466: [OpenMP] Replace math headers with OpenMP wrapper calls

JonChesterfield added a comment. We've got quite a lot of debt in this area and seem at risk of taking on more. Ideally the cuda and hip and openmp headers would be closer to a single header containing: double acosh(double); INSTANTIATE(acosh, cuda_acosh, amdgpu_acosh, intel_acosh);

[PATCH] D111400: [Clang] Implement P2242R3

aaron.ballman added a comment. Herald added a project: All. In D111400#985 , @cor3ntin wrote: > @hubert.reinterpretcast Gentle ping in case you didn't see Aaron's message - > there isn't too much urgency though Another ping on this since a few

[PATCH] D121466: [OpenMP] Replace math headers with OpenMP wrapper calls

tianshilei1992 accepted this revision. tianshilei1992 added a comment. This revision is now accepted and ready to land. LGTM. Support the header file is actually generated automatically. :-) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121466/new/

[PATCH] D121451: [clang-format] Add space to comments starting with '#'.

curdeius added a comment. In D121451#3375229 , @MyDeveloperDay wrote: > Is this conforming to `SpacesInLineCommentPrefix` Yes it is, nothing changed in this regard, I added tests anyway. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D121451: [clang-format] Add space to comments starting with '#'.

curdeius updated this revision to Diff 414669. curdeius added a comment. Add tests for SpacesInLineCommentPrefix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121451/new/ https://reviews.llvm.org/D121451 Files:

[PATCH] D121201: [clang] Merge the SourceRange into ParsedAttributes

aaron.ballman added inline comments. Comment at: clang/include/clang/Sema/ParsedAttr.h:1105 void clearListOnly() { ParsedAttributesView::clearListOnly(); Range = SourceRange(); erichkeane wrote: > This is... oh boy. I'm hopeful you can remove this

[PATCH] D121201: [clang] Merge the SourceRange into ParsedAttributes

aaron.ballman added a reviewer: erichkeane. aaron.ballman added a comment. > I wasn't 100% whether to add the range to ParsedAttributes or even > ParsedAttributesView. I think `ParsedAttributesView` is the correct level for this -- I want to see us tracking the source range for all the parsed

[PATCH] D121201: [clang] Merge the SourceRange into ParsedAttributes

erichkeane added inline comments. Comment at: clang/include/clang/Sema/ParsedAttr.h:920 void clearListOnly() { AttrList.clear(); } Also, this function is now strange/likely needs a rename, since it likely needs to reset the range as well. I believe the

[PATCH] D121201: [clang] Merge the SourceRange into ParsedAttributes

erichkeane added a comment. Just a couple of questions/comments. Otherwise this seems pretty ok. Comment at: clang/lib/Parse/ParseCXXInlineMethods.cpp:745 - ParseGNUAttributeArgs(, LA.AttrNameLoc, Attrs, , + ParseGNUAttributeArgs(, LA.AttrNameLoc, Attrs, nullptr,

[clang] 058c92f - [clang][SVE] Add aarch64-registered-target to sve vector op tests

Author: David Truby Date: 2022-03-11T16:01:00Z New Revision: 058c92f2a4e865d478b29035c207ec0744229d6a URL: https://github.com/llvm/llvm-project/commit/058c92f2a4e865d478b29035c207ec0744229d6a DIFF: https://github.com/llvm/llvm-project/commit/058c92f2a4e865d478b29035c207ec0744229d6a.diff LOG:

[PATCH] D121468: [OpenMP] Add linking of OpenMP math library

jhuber6 created this revision. jhuber6 added reviewers: JonChesterfield, tianshilei1992, jdoerfert, ggeorgakoudis. Herald added subscribers: asavonic, dang, kerbowa, guansong, yaxunl, jvesely. Herald added a project: All. jhuber6 requested review of this revision. Herald added subscribers:

[PATCH] D121466: [OpenMP] Replace math headers with OpenMP wrapper calls

jhuber6 created this revision. jhuber6 added reviewers: JonChesterfield, tianshilei1992, jdoerfert, ggeorgakoudis. Herald added subscribers: asavonic, dang, guansong, yaxunl, mgorny. Herald added a project: All. jhuber6 requested review of this revision. Herald added subscribers: cfe-commits,

[PATCH] D121465: WIP: [clang][modules] Do not report declarations without linkage as ambiguous

jansvoboda11 created this revision. jansvoboda11 added reviewers: Bigcheese, vsapsai. Herald added a project: All. jansvoboda11 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. When finalizing the result of name lookup that encountered

[PATCH] D121451: [clang-format] Add space to comments starting with '#'.

MyDeveloperDay added a comment. Is this conforming to `SpacesInLineCommentPrefix` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121451/new/ https://reviews.llvm.org/D121451 ___ cfe-commits mailing list

[PATCH] D120323: [clang][SVE] Add support for arithmetic operators on SVE types

This revision was automatically updated to reflect the committed changes. Closed by commit rG3aca0ffd5092: [clang][SVE] Add support for arithmetic operators on SVE types (authored by DavidTruby). Changed prior to commit: https://reviews.llvm.org/D120323?vs=413470=414656#toc Repository: rG

[clang] 3aca0ff - [clang][SVE] Add support for arithmetic operators on SVE types

Author: David Truby Date: 2022-03-11T15:39:44Z New Revision: 3aca0ffd5092e37590bbee7e0fcd2b1b5780024c URL: https://github.com/llvm/llvm-project/commit/3aca0ffd5092e37590bbee7e0fcd2b1b5780024c DIFF: https://github.com/llvm/llvm-project/commit/3aca0ffd5092e37590bbee7e0fcd2b1b5780024c.diff LOG:

[PATCH] D121245: [clang][parser] Allow GNU attributes before namespace identifier

aaron.ballman added a comment. Oh wow, GCC has supported this for a long time; I wasn't aware of that. Thanks for this patch! Comment at: clang/lib/Parse/ParseDeclCXX.cpp:79 + + auto ReadLabelAttrubutes = [&] { +// Read label attributes, if present.

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

erichkeane added inline comments. Comment at: llvm/include/llvm/Support/X86TargetParser.def:230 +CPU_SPECIFIC("pentium_m", "pentium-m", 'K', "+cmov,+mmx,+sse,+sse2") +CPU_SPECIFIC("pentium_4_sse3", "", 'L', "+cmov,+mmx,+sse,+sse2,+sse3") +CPU_SPECIFIC("core_2_duo_ssse3", "",

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

erichkeane updated this revision to Diff 414650. erichkeane added a comment. Herald added a project: LLVM. Herald added a subscriber: llvm-commits. add a 'translation' feature to the x86 target so that we can get the 'tune cpu' name from the list. Note that there are 9 with blanks that I was

[PATCH] D121374: [flang][driver] Add support for `-mllvm`

awarzynski updated this revision to Diff 414648. awarzynski added a comment. Fix option spelling in test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121374/new/ https://reviews.llvm.org/D121374 Files: clang/include/clang/Driver/Options.td

[PATCH] D120824: [clang][ASTImporter] Fix a bug when importing CXXDefaultInitExpr.

balazske added a comment. The last unit test is not much different from the lit test, I could not find more simple code to get the wanted result. Is it still good to have the lit test too? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120824/new/

[PATCH] D120824: [clang][ASTImporter] Fix a bug when importing CXXDefaultInitExpr.

balazske updated this revision to Diff 414642. balazske added a comment. Added unit tests. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120824/new/ https://reviews.llvm.org/D120824 Files: clang/lib/AST/ASTImporter.cpp

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

erichkeane added a comment. Thanks all! I'll do some work on populating a list of 'converted names', but I'll definitely need @pengfei and @andrew.w.kaylor help checking the list/filling in what I miss. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121410/new/

[PATCH] D121459: [analyzer] Remove HasAlphaDocumentation tablegen enum value

steakhal created this revision. steakhal added reviewers: aaron.ballman, NoQ, Szelethus, martong. Herald added subscribers: manas, ASDenysPetrov, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, rnkovacs, szepet, baloghadamsoftware, xazax.hun. Herald added a project: All. steakhal requested review

[PATCH] D121457: Clamp negative coverage counters to zero.

Hiralo added a comment. In D121457#3375039 , @lebedev.ri wrote: > Wouldn't it be best to instead fix the non-thread-safety of the coverage > counters? Can you please elaborate or point me to code? Thanks Repository: rG LLVM Github Monorepo

[PATCH] D121457: Clamp negative coverage counters to zero.

lebedev.ri added a comment. Wouldn't it be best to instead fix the non-thread-safety of the coverage counters? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121457/new/ https://reviews.llvm.org/D121457

[PATCH] D121457: Clamp negative coverage counters to zero.

Hiralo created this revision. Hiralo added a reviewer: vsk. Hiralo added projects: clang, LLVM. Herald added subscribers: dexonsmith, wenlei, dang, kbarton, nemanjai. Herald added a project: All. Hiralo requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits. commit

[PATCH] D121206: [AARCH64] ssbs should be enabled by default for cortex-x1, cortex-x1c, cortex-a77

stuij marked 2 inline comments as done. stuij added inline comments. Comment at: clang/test/Preprocessor/aarch64-target-features.c:288 // CHECK-MCPU-CORTEX-A73: "-cc1"{{.*}} "-triple" "aarch64{{.*}}" "-target-feature" "+v8a" "-target-feature" "+fp-armv8" "-target-feature"

[PATCH] D121206: [AARCH64] ssbs should be enabled by default for cortex-x1c

stuij updated this revision to Diff 414635. stuij added a comment. addressed review comments Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D121206/new/ https://reviews.llvm.org/D121206 Files: clang/test/Driver/aarch64-ssbs.c

[PATCH] D121456: [clang-format] Correctly format variable templates.

curdeius created this revision. curdeius added reviewers: MyDeveloperDay, HazardyKnusperkeks, owenpan, cjdb. Herald added a project: All. curdeius requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fixes

[PATCH] D121410: Have cpu-specific variants set 'tune-cpu' as an optimization hint

aaron.ballman added a reviewer: arsenm. aaron.ballman added a subscriber: arsenm. aaron.ballman added a comment. Herald added a subscriber: wdng. Adding @arsenm because of this bit: > Note that the 'valid' list of processors for x86 is in > llvm/include/llvm/Support/X86TargetParser.def. At the

[clang] b44eb20 - [clang-format] Refactor condition in AnnotatingParser::modifyContext(). NFC.

Author: Marek Kurdej Date: 2022-03-11T13:12:43+01:00 New Revision: b44eb207e96a5e20aecbb6084147ae97e3eabd22 URL: https://github.com/llvm/llvm-project/commit/b44eb207e96a5e20aecbb6084147ae97e3eabd22 DIFF: https://github.com/llvm/llvm-project/commit/b44eb207e96a5e20aecbb6084147ae97e3eabd22.diff

[PATCH] D121122: Set FLT_EVAL_METHOD to -1 when fast-math is enabled.

aaron.ballman added inline comments. Comment at: clang/test/CodeGen/eval-method-fast-math.c:83 +#pragma float_control(pop) + +int getFPEvalMethod() { zahiraam wrote: > aaron.ballman wrote: > > What should happen in a case like this? > > ``` > > int

[PATCH] D121455: [clang][dataflow] Add support for nested composite bool expressions

sgatev created this revision. sgatev added reviewers: ymandel, xazax.hun, gribozavr2. Herald added subscribers: tschuett, steakhal, rnkovacs. Herald added a project: All. sgatev requested review of this revision. Herald added a project: clang. This is part of the implementation of the dataflow

[PATCH] D121294: [AArch64][SVE][Clang] Fix crash for incorrect svptrue and svcnt parameters

This revision was automatically updated to reflect the committed changes. Closed by commit rG6c5da880e03c: [AArch64][SVE][Clang] Fix crash for incorrect svptrue and svcnt parameters (authored by MattDevereau). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[clang] 6c5da88 - [AArch64][SVE][Clang] Fix crash for incorrect svptrue and svcnt parameters

Author: Matt Devereau Date: 2022-03-11T11:19:53Z New Revision: 6c5da880e03cafc49008612eb687910f3f805057 URL: https://github.com/llvm/llvm-project/commit/6c5da880e03cafc49008612eb687910f3f805057 DIFF: https://github.com/llvm/llvm-project/commit/6c5da880e03cafc49008612eb687910f3f805057.diff

[PATCH] D121445: [Clang][CSKY] Add the CSKY target and compiler driver

DavidSpickett added inline comments. Comment at: clang/lib/Basic/Targets/CSKY.cpp:43 + Builder.defineMacro("__CSKYABI__", ABI == "abiv2" ? "2" : "1"); + Builder.defineMacro("__cskyabi__", ABI == "abiv2" ? "2" : "1"); + Any need to handle when ABI is not avbiv2

  1   2   >