Page MenuHomePhabricator

lh123 (liu hui)
User

Projects

User does not belong to any projects.

User Details

User Since
Jul 18 2019, 1:05 AM (21 w, 2 d)

Recent Activity

Mon, Dec 9

lh123 added a comment to D71172: [clang][Tooling] Fix potential UB in ExpandResponseFilesCompilationDatabase.

@sammccall Could you commit this, I don't have commit access.

Mon, Dec 9, 1:35 AM · Restricted Project
lh123 updated the diff for D71172: [clang][Tooling] Fix potential UB in ExpandResponseFilesCompilationDatabase.

Address comments.

Mon, Dec 9, 1:35 AM · Restricted Project
lh123 updated the diff for D71172: [clang][Tooling] Fix potential UB in ExpandResponseFilesCompilationDatabase.
Mon, Dec 9, 1:35 AM · Restricted Project

Sun, Dec 8

lh123 created D71172: [clang][Tooling] Fix potential UB in ExpandResponseFilesCompilationDatabase.
Sun, Dec 8, 1:31 AM · Restricted Project

Fri, Nov 29

lh123 added a comment to D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

Thanks a lot for working on this patch, LGTM!

again please make sure the changes are clang-formatted.

I'm sure the code has been formatted.

Fri, Nov 29, 2:57 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

fix typo:
"Could not convert UTF16 To UTF8" -> "Could not convert UTF16 to UTF8"

Fri, Nov 29, 2:55 AM · Restricted Project, Restricted Project
lh123 added inline comments to D70769: [Support] add vfs support for ExpandResponseFiles.
Fri, Nov 29, 1:56 AM · Restricted Project, Restricted Project

Thu, Nov 28

lh123 added inline comments to D70769: [Support] add vfs support for ExpandResponseFiles.
Thu, Nov 28, 4:14 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

address comments.

Thu, Nov 28, 3:06 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

address comments.

Thu, Nov 28, 3:05 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

address comments.

Thu, Nov 28, 2:53 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

fixes no matching constructor for initialization of llvm::StringError.

Thu, Nov 28, 2:29 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

address comments.

Thu, Nov 28, 2:18 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

address comments.

Thu, Nov 28, 2:06 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

fix bug.

Thu, Nov 28, 12:56 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

Sorry, upload the wrong patch.

Thu, Nov 28, 12:56 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.
Thu, Nov 28, 12:21 AM · Restricted Project, Restricted Project

Wed, Nov 27

lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

address comments.

Wed, Nov 27, 11:58 PM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

address comments.

Wed, Nov 27, 11:35 PM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

address comments.

Wed, Nov 27, 10:37 PM · Restricted Project, Restricted Project
lh123 added inline comments to D70769: [Support] add vfs support for ExpandResponseFiles.
Wed, Nov 27, 6:45 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

address comments.

Wed, Nov 27, 6:45 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

rebase

Wed, Nov 27, 5:49 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

address comments.

Wed, Nov 27, 5:41 AM · Restricted Project, Restricted Project
lh123 set the repository for D70769: [Support] add vfs support for ExpandResponseFiles to rG LLVM Github Monorepo.
Wed, Nov 27, 4:51 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

rebase to D70769

Wed, Nov 27, 4:47 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

address comments and delete unused code.

Wed, Nov 27, 4:38 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70769: [Support] add vfs support for ExpandResponseFiles.

Address comments

Wed, Nov 27, 4:29 AM · Restricted Project, Restricted Project
lh123 added a parent revision for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json: D70769: [Support] add vfs support for ExpandResponseFiles.
Wed, Nov 27, 1:46 AM · Restricted Project, Restricted Project
lh123 added a child revision for D70769: [Support] add vfs support for ExpandResponseFiles: D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Wed, Nov 27, 1:46 AM · Restricted Project, Restricted Project
lh123 added a comment to D70769: [Support] add vfs support for ExpandResponseFiles.

Are there any instances where we DON'T want to get the real file system? If not, could the *llvm::vfs::getRealFileSystem() call be put inside cl::ExpandResponseFiles?

Wed, Nov 27, 1:46 AM · Restricted Project, Restricted Project
lh123 created D70769: [Support] add vfs support for ExpandResponseFiles.
Wed, Nov 27, 12:32 AM · Restricted Project, Restricted Project

Tue, Nov 26

lh123 added inline comments to D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Tue, Nov 26, 5:19 AM · Restricted Project, Restricted Project
lh123 added inline comments to D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Tue, Nov 26, 5:10 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

address comments

Tue, Nov 26, 5:01 AM · Restricted Project, Restricted Project
lh123 added inline comments to D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Tue, Nov 26, 4:51 AM · Restricted Project, Restricted Project
lh123 added inline comments to D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Tue, Nov 26, 4:42 AM · Restricted Project, Restricted Project
lh123 updated the summary of D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Tue, Nov 26, 2:53 AM · Restricted Project, Restricted Project

Sat, Nov 23

lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

Expand response files before inferMissingCompileCommands and inferTargetAndDriverMode

Sat, Nov 23, 2:40 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

Address comment

Sat, Nov 23, 12:05 AM · Restricted Project, Restricted Project

Tue, Nov 19

lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

fixes some bug and add more test.

Tue, Nov 19, 6:31 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

fix typo in function document.

Tue, Nov 19, 4:17 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

Address comment

Tue, Nov 19, 4:05 AM · Restricted Project, Restricted Project

Mon, Nov 18

lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

Respect JSONCommandLineSyntax

Mon, Nov 18, 5:10 AM · Restricted Project, Restricted Project
lh123 updated the summary of D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Mon, Nov 18, 4:34 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

Move the implementation to JSONCompilationDatabase.

Mon, Nov 18, 4:34 AM · Restricted Project, Restricted Project
lh123 added a comment to D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

Thanks for taking a look into this, the rsp files issue has came up before in the past but there wasn't enough investment to implement it.

Haven't checked the implementation in detail yet, I believe the layering should be different;

This is a common problem for all of the clang-related tools, as they all share the same "compilation database" abstraction layer, therefore it would be better to implement this at that layer so that other tools (e.g, clang-tidy) can also benefit from this.
You can find the related code in clang/include/clang/Tooling/CompilationDatabase.h and clang/lib/Tooling/CompilationDatabase.cpp.

Also compilation databases has been historically neglecting Virtual File System abstractions, it is hard to change it now. But would be great if you could try to keep that in mind while performing reads.

So would you mind making such changes ?

Mon, Nov 18, 4:08 AM · Restricted Project, Restricted Project
lh123 added inline comments to D70359: [clangd] Show values of more expressions on hover.
Mon, Nov 18, 1:47 AM · Restricted Project
lh123 added inline comments to D70359: [clangd] Show values of more expressions on hover.
Mon, Nov 18, 1:47 AM · Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

format patch

Mon, Nov 18, 12:48 AM · Restricted Project, Restricted Project
lh123 updated the diff for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.

update diff

Mon, Nov 18, 12:45 AM · Restricted Project, Restricted Project

Sun, Nov 17

lh123 added inline comments to D70359: [clangd] Show values of more expressions on hover.
Sun, Nov 17, 12:18 AM · Restricted Project

Nov 14 2019

lh123 added a comment to D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Nov 14 2019, 1:45 AM · Restricted Project, Restricted Project
lh123 added a reviewer for D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json: kadircet.
Nov 14 2019, 12:24 AM · Restricted Project, Restricted Project

Nov 13 2019

lh123 updated the summary of D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Nov 13 2019, 11:11 PM · Restricted Project, Restricted Project
lh123 created D70222: [clang][Tooling] Add support for .rsp files in compile_commands.json.
Nov 13 2019, 11:02 PM · Restricted Project, Restricted Project
lh123 retitled D70164: [Tooling][JSONCompilationDatabase] Fix backslash escaping on Windows from [JSONCompilationDatabase] Fix backslash escaping on Windows to [Tooling][JSONCompilationDatabase] Fix backslash escaping on Windows.
Nov 13 2019, 1:00 AM · Restricted Project
lh123 added inline comments to D70164: [Tooling][JSONCompilationDatabase] Fix backslash escaping on Windows.
Nov 13 2019, 1:00 AM · Restricted Project

Nov 12 2019

lh123 created D70164: [Tooling][JSONCompilationDatabase] Fix backslash escaping on Windows.
Nov 12 2019, 11:11 PM · Restricted Project

Nov 11 2019

lh123 added a comment to D69996: [clangd] Fixes colon escaping on Windows.

I believe this patch fixes the issue, but we need to update the file test/semantic-highlighting.test.ts), the test "Colorizer groups decorations correctly" is diverged from the actual code, although it is still passed (both string and vscode.Uri have the toString method unfortunately).

@lh123 do you mind doing that?

Nov 11 2019, 6:14 AM · Restricted Project, Restricted Project
lh123 created D70078: [clangd] fixes semantic highlighting test.
Nov 11 2019, 6:05 AM · Restricted Project, Restricted Project

Nov 8 2019

lh123 added a comment to D69996: [clangd] Fixes colon escaping on Windows.

Thanks for your reminder, I have sent an email to apply for commit access, but I don't know how long it will take.

Nov 8 2019, 6:53 AM · Restricted Project, Restricted Project
lh123 retitled D69996: [clangd] Fixes colon escaping on Windows from [clangd] Fixed colon escaping on Windows to [clangd] Fixes colon escaping on Windows.
Nov 8 2019, 6:34 AM · Restricted Project, Restricted Project
lh123 added inline comments to D69996: [clangd] Fixes colon escaping on Windows.
Nov 8 2019, 5:31 AM · Restricted Project, Restricted Project
lh123 updated the diff for D69996: [clangd] Fixes colon escaping on Windows.

address comment

Nov 8 2019, 5:28 AM · Restricted Project, Restricted Project
lh123 updated the diff for D69996: [clangd] Fixes colon escaping on Windows.
Nov 8 2019, 3:29 AM · Restricted Project, Restricted Project
lh123 updated the diff for D69996: [clangd] Fixes colon escaping on Windows.

Remove irrelevant files from the patch

Nov 8 2019, 2:52 AM · Restricted Project, Restricted Project
lh123 updated the diff for D69996: [clangd] Fixes colon escaping on Windows.

update diff

Nov 8 2019, 2:49 AM · Restricted Project, Restricted Project
lh123 planned changes to D69996: [clangd] Fixes colon escaping on Windows.

it will cause whitspace escaped problem.

Nov 8 2019, 2:13 AM · Restricted Project, Restricted Project
lh123 created D69996: [clangd] Fixes colon escaping on Windows.
Nov 8 2019, 2:01 AM · Restricted Project, Restricted Project

Nov 7 2019

lh123 added a comment to D69928: [clangd] Set RetainCommentsFromSystemHeaders to true.

LGTM, thanks!
Do you need someone to land this?

Nov 7 2019, 12:31 AM · Restricted Project, Restricted Project

Nov 6 2019

lh123 added a reviewer for D69928: [clangd] Set RetainCommentsFromSystemHeaders to true: kadircet.
Nov 6 2019, 8:50 PM · Restricted Project, Restricted Project
lh123 created D69928: [clangd] Set RetainCommentsFromSystemHeaders to true.
Nov 6 2019, 8:32 PM · Restricted Project, Restricted Project
lh123 added a comment to D69879: [Support] fix mingw-w64 build.

LGTM

Nov 6 2019, 3:35 AM · Restricted Project

Nov 5 2019

lh123 added a comment to D69879: [Support] fix mingw-w64 build.

FWIW, the latest git version of mingw-w64 has this typedef (since llvm started using it here a couple weeks ago), but being able to compile with older versions might be good of course,

Nov 5 2019, 11:19 PM · Restricted Project
lh123 created D69879: [Support] fix mingw-w64 build.
Nov 5 2019, 10:15 PM · Restricted Project
lh123 updated the summary of D69879: [Support] fix mingw-w64 build.
Nov 5 2019, 10:15 PM · Restricted Project

Jul 18 2019

lh123 added a comment to D64970: [clangd] Handle windows line endings in QueryDriver.

LGTM thanks!

Do you have commit access?

Jul 18 2019, 11:21 PM · Restricted Project, Restricted Project, Restricted Project
lh123 retitled D64970: [clangd] Handle windows line endings in QueryDriver from Handle windows line endings in QueryDriver to [clangd] Handle windows line endings in QueryDriver.
Jul 18 2019, 11:02 PM · Restricted Project, Restricted Project, Restricted Project
lh123 updated the diff for D64970: [clangd] Handle windows line endings in QueryDriver.

update diff context.

Jul 18 2019, 10:45 PM · Restricted Project, Restricted Project, Restricted Project
lh123 updated the diff for D64970: [clangd] Handle windows line endings in QueryDriver.

update test.

Jul 18 2019, 10:37 PM · Restricted Project, Restricted Project, Restricted Project
lh123 created D64970: [clangd] Handle windows line endings in QueryDriver.
Jul 18 2019, 10:21 PM · Restricted Project, Restricted Project, Restricted Project