[PATCH] D65153: [clangd] Also accept flags from CLANGD_FLAGS variable.

2019-07-25 Thread Sam McCall via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL366991: [clangd] Also accept flags from CLANGD_FLAGS variable. (authored by sammccall, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Changed prior to commit: htt

[PATCH] D65153: [clangd] Also accept flags from CLANGD_FLAGS variable.

2019-07-25 Thread Haojian Wu via Phabricator via cfe-commits
hokein accepted this revision. hokein added inline comments. This revision is now accepted and ready to land. Comment at: clang-tools-extra/clangd/test/log.test:9 +CHECK: Starting LSP over stdin/stdout + nit: remove the extra trailing blank line? Repository:

[PATCH] D65153: [clangd] Also accept flags from CLANGD_FLAGS variable.

2019-07-23 Thread Sam McCall via Phabricator via cfe-commits
sammccall created this revision. sammccall added a reviewer: kadircet. Herald added subscribers: cfe-commits, arphaman, jkorous, MaskRay, ilya-biryukov. Herald added a project: clang. This simplifies various workflows, particularly in debugging/development. e.g. editors will tend to propagate fla