[PATCH] D52778: [Preprocessor] Hide include typo correction behind SpellChecking.

2018-10-02 Thread Haojian Wu via Phabricator via cfe-commits
hokein updated this revision to Diff 167939. hokein added a comment. Fix review comment. Repository: rC Clang https://reviews.llvm.org/D52778 Files: lib/Lex/PPDirectives.cpp Index: lib/Lex/PPDirectives.cpp === --- lib/Lex/PP

[PATCH] D52778: [Preprocessor] Hide include typo correction behind SpellChecking.

2018-10-02 Thread Haojian Wu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. hokein marked an inline comment as done. Closed by commit rC343591: [Preprocessor] Hide include typo correction behind SpellChecking. (authored by hokein, committed by ). Changed prior to commit: https://reviews.llvm.org/

[PATCH] D52778: [Preprocessor] Hide include typo correction behind SpellChecking.

2018-10-02 Thread Sam McCall via Phabricator via cfe-commits
sammccall accepted this revision. sammccall added inline comments. This revision is now accepted and ready to land. Comment at: lib/Lex/PPDirectives.cpp:1891 StringRef OriginalFilename = Filename; - if (!File) { -// A heuristic to correct a typo file name by r

[PATCH] D52778: [Preprocessor] Hide include typo correction behind SpellChecking.

2018-10-02 Thread Haojian Wu via Phabricator via cfe-commits
hokein created this revision. hokein added a reviewer: sammccall. Similar to Sema typo correction, the Preprocessor typo correction should also be hidden behind the SpellChecking flag. Repository: rC Clang https://reviews.llvm.org/D52778 Files: lib/Lex/PPDirectives.cpp Index: lib/Lex/PPD