Timm =?utf-8?q?Bäder?= <tbae...@redhat.com>, Timm =?utf-8?q?Bäder?= <tbae...@redhat.com>, Timm =?utf-8?q?Bäder?= <tbae...@redhat.com>, Timm =?utf-8?q?Bäder?= <tbae...@redhat.com>, Timm =?utf-8?q?Bäder?= <tbae...@redhat.com>, Timm =?utf-8?q?Bäder?= <tbae...@redhat.com>, Timm =?utf-8?q?Bäder?= <tbae...@redhat.com>, Timm =?utf-8?q?Bäder?= <tbae...@redhat.com> Message-ID: In-Reply-To: <llvm/llvm-project/pull/66514/cl...@github.com>
tbaederr wrote: > Also, redirecting output to a file doesn't always disable color output. From > our precommit CI pipeline: > https://buildkite.com/llvm-project/clang-ci/builds/4334#018af546-9a63-45f1-a8f9-6262f3d45b75 > -- click on the raw log and you'll see it has contents like: > > ``` > �_bk;t=1696332172990�~~~ Running global pre-checkout hook > �_bk;t=1696332172991��[90m$�[0m /etc/buildkite-agent/hooks/pre-checkout > �_bk;t=1696332173007�BUILDKITE_REPO: > https://github.com/llvm-premerge-tests/llvm-project.git > > �_bk;t=1696332173008�current remote URL: > https://github.com/llvm-premerge-tests/llvm-project.git > > �_bk;t=1696332173010�GC counter 16/20 > > �_bk;t=1696332173026��[90m$�[0m cd > /var/lib/buildkite-agent/builds/linux-56-7f758798dd-v9p4k-1/llvm-project/clang-ci > �_bk;t=1696332173026��[90m# OLDPWD added�[0m > �_bk;t=1696332173026��[90m# PWD changed�[0m > ``` Is that on purpose? > Did you investigate what Richard was suggesting with caching "we should be > back to normal lexing here" information and re-lexing from that point? That > would make me feel a lot more comfortable than re-lexing the entire file. I am looking at it, but I'm not certain where to get the information. I definitely want to avoid adding overhead to regular, diagnostic-less lexing. https://github.com/llvm/llvm-project/pull/66514 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
