[all-commits] [llvm/llvm-project] 22a7f7: [symbolizer] Empty string is not an error (#92660)

Serge Pavlov via All-commits all-commits at lists.llvm.org
Mon Jun 3 04:21:06 PDT 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 22a7f7c3314328dd1ac20042158f4b68a4ac5c1f
      https://github.com/llvm/llvm-project/commit/22a7f7c3314328dd1ac20042158f4b68a4ac5c1f
  Author: Serge Pavlov <sepavloff at gmail.com>
  Date:   2024-06-03 (Mon, 03 Jun 2024)

  Changed paths:
    M llvm/test/tools/llvm-symbolizer/get-input-file.test
    M llvm/tools/llvm-symbolizer/llvm-symbolizer.cpp

  Log Message:
  -----------
  [symbolizer] Empty string is not an error (#92660)

After commit 1792852f86dc7 ([symbolizer] Change reaction on invalid
input) llvm-symbolizer issues an error on malformed command instead of
echoing it to the standard output, as in previous versions. It turns out
this behavior broke a use case when echoing was used to check if
llvm-symbolizer is working (https://github.com/llvm/llvm-project/commit/1792852f86dc75efa1f44d46b1a0daf386d64afa#commitcomment-142161925).

With this change an empty line as input is not considered as an error
anymore and does not produce any output on stderr. llvm-symbolizer still
respond on empty line with line not found, this is consistent with GNU
addr2line.

---------

Co-authored-by: James Henderson <46713263+jh7370 at users.noreply.github.com>



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list