This revision was automatically updated to reflect the committed changes.
Closed by commit rGd3e14fafc69a: [analyzer][NFC] Display the correct function
name even in crash dumps (authored by steakhal).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105
Author: Balazs Benics
Date: 2021-07-12T09:06:46+02:00
New Revision: d3e14fafc69a07e3dab9ddb91f1d810bb5f8d7a0
URL:
https://github.com/llvm/llvm-project/commit/d3e14fafc69a07e3dab9ddb91f1d810bb5f8d7a0
DIFF:
https://github.com/llvm/llvm-project/commit/d3e14fafc69a07e3dab9ddb91f1d810bb5f8d7a0.diff
StephenFan added inline comments.
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:115
+ return make_filter_range(SupportedExtensionInfos,
+ [&](const RISCVSupportedExtensionInfo &ExtInfo) {
+ return ExtInfo.Name == ExtName;
---
jdoerfert reopened this revision.
jdoerfert added a comment.
Herald added a subscriber: dexonsmith.
> A pthread_create() related test is XFAIL-ed, as it relied on it being
> recognized as a builtin based on its name.
> The builtin declaration syntax is too restrictive and doesn't allow custom
>
nridge updated this revision to Diff 357827.
nridge added a comment.
Rebase on top of refactor patch changes
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104619/new/
https://reviews.llvm.org/D104619
Files:
clang/lib/AST/TypePrinter.cpp
clang/
nridge added inline comments.
Comment at: clang/unittests/AST/DeclPrinterTest.cpp:59
+ return PrintedNodeMatches(Code, Args, NodeMatch, ExpectedPrinted,
+ FileName, PrinterType{PrintDecl},
+ PolicyModifier, AllowE
nridge updated this revision to Diff 357826.
nridge marked 3 inline comments as done.
nridge added a comment.
Address review comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105457/new/
https://reviews.llvm.org/D105457
Files:
clang/unittes
chh added inline comments.
Comment at: clang-tools-extra/clang-tidy/ClangTidy.cpp:310
+struct DeclFilter {
+ DeclFilter(ClangTidyContext &Ctx, SourceManager &SM)
+ : Context(Ctx), Sources(SM) {
sammccall wrote:
> Ctx is ultimately unused, just take the rege
chh updated this revision to Diff 357824.
chh marked 7 inline comments as done.
chh added a comment.
sync up latest clang/llvm source; more format/style changes
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98710/new/
https://reviews.llvm.org/D98710
Files:
clang-tools-extra/clang-tid
nridge marked an inline comment as done.
nridge added inline comments.
Comment at: clang-tools-extra/clangd/index/SymbolCollector.cpp:159
+// indexed, we walk further up because Ref::Container should always be
+// an indexed symbol.
+const Decl *getRefContainer(const Decl *Enclos
nridge updated this revision to Diff 357823.
nridge added a comment.
Address review comments
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D105083/new/
https://reviews.llvm.org/D105083
Files:
clang-tools-extra/clangd/index/SymbolCollector.cpp
c
hubert.reinterpretcast added inline comments.
Comment at: clang/test/CodeGenCXX/cxx11-thread-local-reference.cpp:13
// LINUX: @_ZTH1r ={{.*}} alias void (), void ()* @__tls_init
+// AIXX: @_ZTH1r ={{.*}} alias void (), void ()* @__tls_init
// DARWIN: @_ZTH1r = internal alias vo
nemanjai requested changes to this revision.
nemanjai added a comment.
This revision now requires changes to proceed.
I believe that your failing test case is because you are attempting to emit
code for these builtins in the target independent code and the values just
happen to match up.
luismarques added a comment.
This patch is nearly there! Just address the remaining review comments and it
LGTM.
BTW, please mark all addressed inline comments as done. I think a few were
missed, and it's helpful for a large patch like this.
Comment at: llvm/lib/Target/RISCV
luismarques accepted this revision.
luismarques 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/D98113/new/
https://reviews.llvm.org/D98113
__
jrtc27 added a comment.
Ping?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D98113/new/
https://reviews.llvm.org/D98113
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-b
Author: Vassil Vassilev
Date: 2021-07-11T14:40:10Z
New Revision: 5922f234c8c95f61534160a31db15dfc10da9b60
URL:
https://github.com/llvm/llvm-project/commit/5922f234c8c95f61534160a31db15dfc10da9b60
DIFF:
https://github.com/llvm/llvm-project/commit/5922f234c8c95f61534160a31db15dfc10da9b60.diff
LO
Ericson2314 updated this revision to Diff 357643.
Ericson2314 added a comment.
Herald added a subscriber: whisperity.
rebased, fixed some new DESTINATION
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99484/new/
https://reviews.llvm.org/D99484
File
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG6775fc6ffa3c: [clang-repl] Implement partial translation
units and error recovery. (authored by v.g.vassilev).
Herald added a project: clang.
Reposi
Author: Vassil Vassilev
Date: 2021-07-11T10:23:41Z
New Revision: 6775fc6ffa3ca1c36b20c25fa4e7f48f81213cf2
URL:
https://github.com/llvm/llvm-project/commit/6775fc6ffa3ca1c36b20c25fa4e7f48f81213cf2
DIFF:
https://github.com/llvm/llvm-project/commit/6775fc6ffa3ca1c36b20c25fa4e7f48f81213cf2.diff
LO
20 matches
Mail list logo