[all-commits] [llvm/llvm-project] 5ce141: [symbolizer] Empty string is not an error (#97781)

[Serge Pavlov via All-commits]

  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 5ce141f7af7ba6a42dac41562fb06160a7863b59
      https://github.com/llvm/llvm-project/commit/5ce141f7af7ba6a42dac41562fb06160a7863b59
  Author: Serge Pavlov <sepavloff at gmail.com>
  Date:   2024-07-05 (Fri, 05 Jul 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 (#97781)

This is recommit of llvm/llvm-project#92660, reverted in
llvm/llvm-project#94424.
Original commit message is below.

After commit
https://github.com/llvm/llvm-project/commit/1792852f86dc75efa1f44d46b1a0daf386d64afa
([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.



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