[PATCH] D82103: [analyzer] Remove forbidden characters from a SourceLocation filename for a graph dump on Windows
Denys Petrov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Jun 19 13:37:04 PDT 2020
ASDenysPetrov marked an inline comment as done.
ASDenysPetrov added inline comments.
================
Comment at: clang/utils/analyzer/exploded-graph-rewriter.py:383
+ # when directory name starts with the letter `l`.
+ if sys.platform == 'win32':
+ # Find all `\l` (like `,\l`, `}\l`, `[\l`) except `\\l`,
----------------
NoQ wrote:
> I think this too doesn't need to be platform specific(?)
Oh, so sorry! This is a mess. It somehow got here from my another patch. I'll correct it. This should not be here.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D82103/new/
https://reviews.llvm.org/D82103
More information about the cfe-commits
mailing list