bkramer (Benjamin Kramer)
Senior Procrastinator

Projects

User does not belong to any projects.

User Details

User Since
Jul 10 2012, 1:37 PM (271 w, 3 d)

d0k

Recent Activity

Wed, Sep 20

bkramer accepted D38087: [clang-tidy] Fix example in documentation, NFC.

fun fun fun fun fun

Wed, Sep 20, 11:23 AM

Wed, Sep 13

bkramer accepted D37564: Update users of llvm::sys::ExecuteAndWait etc..

lgtm too

Wed, Sep 13, 9:45 AM
bkramer accepted D37563: Convenience/safety fix for llvm::sys::Execute(And|No)Wait.

lgtm

Wed, Sep 13, 8:29 AM

Fri, Sep 8

bkramer accepted D37382: Fixed a crash in code completion..

lg

Fri, Sep 8, 6:12 AM

Tue, Sep 5

bkramer edited reviewers for D37474: [PCH] Allow VFS to be used for tests that generate PCH files, added: ilya-biryukov; removed: cfe-commits.
Tue, Sep 5, 7:29 AM

Mon, Sep 4

bkramer accepted D37435: Don't show deleted function (constructor) candidates for code completion.

lgtm

Mon, Sep 4, 7:58 AM
bkramer added inline comments to D37435: Don't show deleted function (constructor) candidates for code completion.
Mon, Sep 4, 7:10 AM

Aug 21 2017

bkramer accepted D36821: [libclang]: Honor LIBCLANG_NOTHREADS for clang_parseTranslationUnit*.

lg

Aug 21 2017, 3:41 AM
bkramer accepted D36872: Fixed a crash on replaying Preamble's PP conditional stack..

This looks good to me.

Aug 21 2017, 3:20 AM

Aug 14 2017

bkramer accepted D36397: [clangd] Fixed a data race..

I think this can be committed now. It's still a bit awkward but we can address that later.

Aug 14 2017, 12:59 AM

Aug 9 2017

bkramer added a comment to D36458: Fix crash when current lexer is nullptr.

Changes like this should come with a small, c-index-test based, test case so we don't reintroduce the same bug in the future.

Aug 9 2017, 5:04 AM

Aug 8 2017

bkramer added inline comments to D36390: Fix overloaded static functions in SemaCodeComplete.
Aug 8 2017, 7:00 AM
bkramer accepted D36330: Updated two annotations for Store.h and CodeGenFunction.h..

lg

Aug 8 2017, 6:45 AM
bkramer added a comment to D36458: Fix crash when current lexer is nullptr.

Test case?

Aug 8 2017, 4:41 AM
bkramer added a reviewer for D36458: Fix crash when current lexer is nullptr: erikjv.
Aug 8 2017, 4:41 AM

Aug 7 2017

bkramer added a comment to D36187: [clang-diff] Use the relative name for NamedDecls.

There's some similar code in tools/clang/lib/Tooling/Core/Lookup.cpp, it might make sense to share it. Otherwise this looks good.

Aug 7 2017, 6:47 AM
bkramer added inline comments to D36390: Fix overloaded static functions in SemaCodeComplete.
Aug 7 2017, 6:25 AM
bkramer added inline comments to D36330: Updated two annotations for Store.h and CodeGenFunction.h..
Aug 7 2017, 6:19 AM

Aug 3 2017

bkramer accepted D36302: [msan] Switch the pvalloc overflow test to a lit test.

lgtm

Aug 3 2017, 11:28 PM

Aug 2 2017

bkramer accepted D36154: Adapt clang-tidy checks to changing semantics of hasDeclaration..

looks good

Aug 2 2017, 1:21 AM

Aug 1 2017

bkramer added inline comments to D36154: Adapt clang-tidy checks to changing semantics of hasDeclaration..
Aug 1 2017, 9:10 AM
bkramer accepted D36133: [clangd] Rewrote AST and Preamble management..

ship it!

Aug 1 2017, 8:34 AM
bkramer added inline comments to D36133: [clangd] Rewrote AST and Preamble management..
Aug 1 2017, 3:21 AM

Jul 31 2017

bkramer accepted D36095: [clangd] Allow to get vfs::FileSystem used inside codeComplete..

lg (it could use a test case though)

Jul 31 2017, 10:00 AM

Jul 28 2017

bkramer accepted D27104: Unify and simplify the behavior of the hasDeclaration matcher..

lg under the precondition that clang-tidy tests still work.

Jul 28 2017, 4:19 AM

Jul 25 2017

bkramer accepted D35828: Fix incorrect use of current directory to find moved paths in ASTReader..

lg

Jul 25 2017, 3:12 AM

Jul 19 2017

bkramer accepted D35617: [clangd] Allow specifying -resource-dir.

Looks good.

Jul 19 2017, 7:48 AM · Restricted Project

Jul 10 2017

bkramer accepted D35203: Avoid white spaces in file names. NFC.

This is fine. We should fix phab though.

Jul 10 2017, 9:03 AM
bkramer accepted D35206: [LLVM] Get rid of white spaces in file names in a DebugInfo test. NFC.

This is fine. We should fix phab though.

Jul 10 2017, 9:03 AM

Jun 30 2017

bkramer accepted D34882: Fix invalid warnings for header guards in preambles.

This fixes the issues I'm seeing.

Jun 30 2017, 8:54 AM
bkramer edited reviewers for D34304: Allow CompilerInvocations to generate .d files., added: klimek; removed: bkramer.
Jun 30 2017, 3:16 AM

Jun 6 2017

bkramer accepted D33930: Do not pick up by default the LLVM style if passing -format.

Looks good, but I'm wondering if we should also change the default in clang-apply-replacements itself.

Jun 6 2017, 5:29 AM

May 30 2017

bkramer accepted D33678: [clangd] Mark results of clangd requests with a tag provided by the FileSystemProvider..

lg

May 30 2017, 8:07 AM · Restricted Project

May 24 2017

bkramer accepted D33416: [clangd] Allow to use vfs::FileSystem for file accesses..

Looks good. Make sure to watch the windows buildbots after submitting.

May 24 2017, 5:48 AM

May 23 2017

bkramer added inline comments to D33416: [clangd] Allow to use vfs::FileSystem for file accesses..
May 23 2017, 2:45 AM
bkramer accepted D33397: Allow to use vfs::FileSystem for file accesses inside ASTUnit..

lg

May 23 2017, 2:28 AM

May 22 2017

bkramer accepted D33395: [clangd] Split clangd into library+executable (mainly for unit tests)..

looks good to me

May 22 2017, 9:27 AM · Restricted Project

May 19 2017

bkramer edited reviewers for D33045: [libclang] Avoid more stats than necessary for reparse., added: ilya-biryukov; removed: cfe-commits.
May 19 2017, 7:07 AM
bkramer accepted D33042: [libclang] Allow to suspend a translation unit..

This makes sense to me.

May 19 2017, 7:05 AM
bkramer accepted D33350: [clangd] Switch to incomplete translation units.

lg

May 19 2017, 3:05 AM

May 17 2017

bkramer accepted D33270: [Frontend] Remove unused TemporaryFiles.

Kill it :)

May 17 2017, 7:58 AM
bkramer accepted D33273: Recommit "[include-fixer] Don't throw exception when parsing unknown ar… …guments in vim script.".

looks good, thanks!

May 17 2017, 5:50 AM

May 15 2017

bkramer accepted D33047: [ClangD] Refactor clangd into separate components.

I believe this is good enough now.

May 15 2017, 6:43 AM · Restricted Project

May 12 2017

bkramer accepted D33103: [clang-tidy] TwineLocalCheck: add param # checking.

looks good, thanks!

May 12 2017, 1:19 AM · Restricted Project

Apr 28 2017

bkramer added a comment to D30946: [ScopePrinting] Added support to print full scopes of types and declarations..

Can you please run clang-format on this change? There are pieces that don't follow the style.

Apr 28 2017, 2:45 AM
bkramer added a reviewer for D30946: [ScopePrinting] Added support to print full scopes of types and declarations.: rsmith.
Apr 28 2017, 2:44 AM

Apr 21 2017

bkramer added a comment to D32234: [Clangd] Support Authority-less URIs.

Still looking good. Will commit this soon.

Apr 21 2017, 8:52 AM · Restricted Project
bkramer added a comment to D32351: [Tooling][libclang] Remove unused CompilationDatabase::MappedSources.

lg from my side. It would be good to wait until Manuel is back though, I think he had plans for extending this interface at some point.

Apr 21 2017, 8:43 AM
bkramer accepted D32238: [Clangd] Failed to decode params using 1.x-compatible request message.

lg

Apr 21 2017, 7:10 AM · Restricted Project

Apr 19 2017

bkramer accepted D32234: [Clangd] Support Authority-less URIs.

This is fine. Test case would be nice though.

Apr 19 2017, 12:59 PM · Restricted Project
bkramer added a comment to D32238: [Clangd] Failed to decode params using 1.x-compatible request message.

A test case would be nice.

Apr 19 2017, 12:59 PM · Restricted Project

Mar 30 2017

bkramer accepted D31328: [clangd] Add code completion support.

lg

Mar 30 2017, 6:17 AM

Mar 28 2017

bkramer accepted D27810: FileManager: mark virtual file entries as valid entries.

lg

Mar 28 2017, 2:23 AM

Mar 27 2017

bkramer accepted D30367: CodeGen : Check LLVM_ENABLE_DUMP definition for dumpMachineInstrRangeWithSlotIndex..

This is fine.

Mar 27 2017, 1:38 AM
bkramer accepted D30248: [libclang] Fix crash in member access code completion with implicit base.

Let's fix those crashes one at a time. This patch looks good, only minor comments below.

Mar 27 2017, 1:35 AM

Mar 24 2017

bkramer added inline comments to D31328: [clangd] Add code completion support.
Mar 24 2017, 7:01 AM

Mar 23 2017

bkramer accepted D31288: [libclang] Bury dead TemporaryFiles.

kill it with fire

Mar 23 2017, 9:18 AM

Mar 16 2017

bkramer added a comment to D31019: [clangd] [RFC] Use libclang and CXTranslationUnit instead of ASTUnit.

libclang is supposed to be used for bindings to other languages (it's a pure C interface) and its stable. ASTUnit is the C++ side of libclang, libclang is just a thin wrapper around ASTUnit. ASTUnit itself suffers from lots of technical debt because clang changed since ASTUnit was created and ASTUnit was never refactored to accommodate that.

Mar 16 2017, 9:36 AM
bkramer resigned from D31019: [clangd] [RFC] Use libclang and CXTranslationUnit instead of ASTUnit.

It's not. the goal is to get rid of ASTUnit inside of clangd in the long term as it's a big problem for extensibility. libclang is just a wrapper for ASTUnit, with even more problems.

Mar 16 2017, 3:52 AM

Mar 15 2017

bkramer accepted D30975: [msan] Relaex lit check in wcsncpy.cc.

lg

Mar 15 2017, 5:22 AM

Mar 14 2017

bkramer accepted D30675: [clangd] Fix not being able to attach a debugger on macOS.

lg, do you have commit access?

Mar 14 2017, 1:32 PM

Mar 13 2017

bkramer accepted D30720: [include-fixer] Add fuzzy SymbolIndex, where identifier needn't match exactly..

lg as a prototype.

Mar 13 2017, 9:00 AM

Mar 8 2017

bkramer accepted D30685: [include-fixer] Remove line number from Symbol identity.

lg

Mar 8 2017, 9:19 AM
bkramer added a comment to D30675: [clangd] Fix not being able to attach a debugger on macOS.

Generally makes sense. Is there any reason for the #ifdef? Windows has errno and EINTR too.

Mar 8 2017, 4:45 AM
bkramer added a comment to D27810: FileManager: mark virtual file entries as valid entries.

I assume this is fine but I don't really understand what's going on. A test case would be great.

Mar 8 2017, 3:29 AM

Mar 1 2017

bkramer updated the diff for D30498: [clangd] Add support for FixIts..
  • Use typedef instead of decltype()
  • Rename local variable not to shadow member.
  • Give FixIts their own mutex
  • Use const instead of let in typescript
Mar 1 2017, 8:07 AM
bkramer added a comment to D30367: CodeGen : Check LLVM_ENABLE_DUMP definition for dumpMachineInstrRangeWithSlotIndex..

This will get you an unused function warning if NDEBUG && LLVM_ENABLE_DUMP. You can use LLVM_DUMP_METHOD to avoid that.

Mar 1 2017, 7:39 AM
bkramer created D30498: [clangd] Add support for FixIts..
Mar 1 2017, 6:34 AM
bkramer accepted D30102: [clangd] Add a toy VS Code integration for development purposes.

lgtm

Mar 1 2017, 6:01 AM

Feb 27 2017

bkramer accepted D30425: [clangd] Make clangd install to bin.

lg

Feb 27 2017, 1:50 PM
bkramer accepted D30210: [include-fixer] Add usage count to find-all-symbols..

lg

Feb 27 2017, 1:46 PM
bkramer accepted D30399: clang-format: [JS] whitespace after async in arrow functions..

lg, but I don't really know JS ;)

Feb 27 2017, 2:34 AM

Feb 21 2017

bkramer created D30191: [clang-tidy] Reword the "code outside header guard" warning..
Feb 21 2017, 3:01 AM

Feb 16 2017

bkramer accepted D29990: [clangd] Implement format on type.

lg

Feb 16 2017, 2:26 AM

Feb 15 2017

bkramer added inline comments to D29990: [clangd] Implement format on type.
Feb 15 2017, 10:17 AM
bkramer updated the diff for D29886: [clangd] Wire up ASTUnit and publish diagnostics with it..
  • Inline the request struct again.
Feb 15 2017, 6:54 AM
bkramer updated the diff for D29886: [clangd] Wire up ASTUnit and publish diagnostics with it..
  • Do not lock while running DocumentStore callbacks
  • Replace fake queue with a real queue (but it still has at most one element.
Feb 15 2017, 6:37 AM
bkramer added inline comments to D29886: [clangd] Wire up ASTUnit and publish diagnostics with it..
Feb 15 2017, 5:42 AM
bkramer updated the diff for D29886: [clangd] Wire up ASTUnit and publish diagnostics with it..
  • Address review comments.
Feb 15 2017, 5:42 AM

Feb 14 2017

bkramer accepted D29755: Cache FileID when translating diagnostics in PCH files.

lg

Feb 14 2017, 6:34 AM
bkramer updated the diff for D29899: [clang-tidy] Add support for NOLINTNEXTLINE..
  • Simplify code by not worrying about \r
  • Don't allow blank lines between NOLINTNEXTLINE and the warning
Feb 14 2017, 4:58 AM

Feb 13 2017

bkramer created D29899: [clang-tidy] Add support for NOLINTNEXTLINE..
Feb 13 2017, 10:17 AM
bkramer created D29886: [clangd] Wire up ASTUnit and publish diagnostics with it..
Feb 13 2017, 2:29 AM

Feb 6 2017

bkramer updated the diff for D29451: Add a prototype for clangd v0.1.

Address review comments. Make test actually run (missing cmake file)

Feb 6 2017, 7:08 AM
bkramer added a comment to D29451: Add a prototype for clangd v0.1.

This might be a bad question, but is there any particular reason why you didn't use the YAML Traits API for parsing instead of the raw YAML Stream API? In my experience the traits based API is quite convenient for structured data like various options and parameters in this protocol.

Feb 6 2017, 7:08 AM

Feb 2 2017

bkramer accepted D29460: [change-namespace] fix unscoped enum constant references..

lg

Feb 2 2017, 9:50 AM
bkramer accepted D29447: [change-namespace] check using shadow decl correctly when shortening namespace specifiers..

lg

Feb 2 2017, 7:27 AM
bkramer created D29451: Add a prototype for clangd v0.1.
Feb 2 2017, 6:33 AM

Jan 30 2017

bkramer accepted D29303: In VirtualCallChecker, handle indirect calls.

lg

Jan 30 2017, 2:17 PM
bkramer added a comment to D29303: In VirtualCallChecker, handle indirect calls.

Your test case is fine, it crashes with assertions enabled.

Jan 30 2017, 1:40 PM

Jan 26 2017

bkramer accepted D29182: [change-namespace] correctly shorten namespace when references have leading '::'.

lg

Jan 26 2017, 8:41 AM
bkramer accepted D29176: [change-namespace] add leading '::' to references in new namespace when name conflict is possible..

Only nits below.

Jan 26 2017, 7:06 AM

Jan 25 2017

bkramer added a comment to D29124: [ARM] GlobalISel: Fix stack-use-after-scope bug..

Though, hum, hard to make tests for that...

Jan 25 2017, 6:50 AM
bkramer accepted D29124: [ARM] GlobalISel: Fix stack-use-after-scope bug..

lg

Jan 25 2017, 6:37 AM

Jan 20 2017

bkramer accepted D28974: [Basic] Remove the 32-bit x86 and ARM targets for Fuchsia.

lg

Jan 20 2017, 2:56 PM
bkramer accepted D28941: Remove 'RUN: false' from Driver/response-file.c.

lg. I assume the test passes with the RUN: false removed?

Jan 20 2017, 3:08 AM

Jan 13 2017

bkramer accepted D28334: [clang-tidy] Add -extra-arg and -extra-arg-before to run-clang-tidy.py.

This looks useful to me.

Jan 13 2017, 5:31 AM
bkramer accepted D28144: clang support for Mageia 6 distro.

lgtm

Jan 13 2017, 5:30 AM
bkramer added a comment to D28669: [ARM] Remove ubig32_t cast from ARMAttributeParser.

can we just use support::endian::read32le and support::endian::read32be instead of the casting magic? They do the same thing but should have no problem with implicit conversions in a ternary expression.

Jan 13 2017, 4:52 AM

Jan 11 2017

bkramer accepted D27920: [find-all-symbols] Index partial template specializations..

The reason why we disallowed specializations is that there are type-trait classes that get specialized all over the place (DenseMapInfo in LLVM is one example). I think we can get away with allowing partial specializations though. Those should be much less common and we can give the user a choice between the headers that contain them.

Jan 11 2017, 2:37 AM