RKSimon added a comment.
(post) LGTM
Repository:
rL LLVM
https://reviews.llvm.org/D44354
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
RKSimon updated this revision to Diff 403574.
RKSimon added a comment.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
update lldb docs as well
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114639/new/
https://reviews.llvm.or
RKSimon updated this revision to Diff 403924.
RKSimon added a comment.
update lldb cmake command
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114639/new/
https://reviews.llvm.org/D114639
Files:
clang/docs/ClangFormatStyleOptions.rst
clang/doc
RKSimon updated this revision to Diff 404076.
RKSimon added a comment.
fix cut+paste typo
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114639/new/
https://reviews.llvm.org/D114639
Files:
clang/docs/ClangFormatStyleOptions.rst
clang/docs/Users
RKSimon added a comment.
@stella.stamenova It looks like the mlir-windows buildbot is still using VS2017:
CMake Error at cmake/modules/CheckCompilerVersion.cmake:39 (message):
Host Visual Studio version must be at least 19.20, your version is
19.16.27045.0.
Are you able to make the fix
This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG058c5dfc78cd: Raise the minimum Visual Studio version to
VS2019 (authored by RKSimon).
Changed prior to commit:
https://reviews.llvm.org/D114639?v
RKSimon added a comment.
mlir-windows buildbot looks to be green now - thanks!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D114639/new/
https://reviews.llvm.org/D114639
___
lldb-commits mailing list
lld
RKSimon added inline comments.
Herald added a subscriber: JDevlieghere.
Comment at: llvm/include/llvm/DebugInfo/CodeView/CVTypeVisitor.h:12
+#include "llvm/ADT/ArrayRef.h"
#include "llvm/DebugInfo/CodeView/CVRecord.h"
Forward declare ArrayRef<> ?
Repository:
RKSimon added a comment.
Please add a reference to
https://discourse.llvm.org/t/include-what-you-use-include-cleanup/5831 in the
summary
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121168/new/
https://reviews.llvm.org/D121168
RKSimon added a comment.
@serge-sans-paille Any luck with tracking down
https://lab.llvm.org/buildbot/#/builders/67/builds/6136 ?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121168/new/
https://reviews.llvm.org/D121168
_
RKSimon added a comment.
@ayermolo This has broken some buildbots - please can you take a look?
https://lab.llvm.org/buildbot/#/builders/245/builds/3279
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D139379/new/
https://reviews.llvm.org/D139379
__
RKSimon added a comment.
@haampie @JDevlieghere This is failing on several buildbots please can you take
a look :
http://lab.llvm.org:8011/builders/clang-x86_64-debian-new-pass-manager-fast/builds/13553
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org
12 matches
Mail list logo