[PATCH] D117570: Handle whitespace in symbol list

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 18 09:52:39 PST 2022


MaskRay added a comment.

> No test provided because test/Transforms/SampleProfile/uncompressed-profile-symbol-list.ll fails on Windows if the git checkout uses CRLF line endings.

You may just call `SymbolStr.trim()` (see `lld/Common/Args.cpp) and add a test with whitespace.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117570/new/

https://reviews.llvm.org/D117570



More information about the llvm-commits mailing list