[PATCH] D75390: Fix GSYM tests to run the yaml files and fix test failures on some machines.

Greg Clayton via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Mar 4 19:35:41 PST 2020


clayborg added a comment.

Recommitted with

commit ffe6695acf1f011a2acda31624beee826fb68956 <https://reviews.llvm.org/rGffe6695acf1f011a2acda31624beee826fb68956> (HEAD -> master, origin/master, origin/HEAD)
Author: Greg Clayton <gclayton at fb.com>
Date:   Wed Mar 4 19:28:24 2020 -0800

  Fix buildbots with merge that didn't happen for 4050b01ba9ece02721ec496383baee219ca8cc2b.

commit 4050b01ba9ece02721ec496383baee219ca8cc2b <https://reviews.llvm.org/rG4050b01ba9ece02721ec496383baee219ca8cc2b>
Author: Greg Clayton <gclayton at fb.com>
Date:   Wed Mar 4 19:12:08 2020 -0800

  Fix GSYM tests to run the yaml files and fix test failures on some machines.
  
  YAML files were not being run during lit testing as there was no lit.local.cfg file. Once this was fixed, some buildbots would fail due to a StringRef that pointed to a std::string inside of a temporary llvm::Triple object. These issues are fixed here by making a local triple object that stays around long enough so the StringRef points to valid data. Fixed memory sanitizer bot bugs as well.
  
  Differential Revision: https://reviews.llvm.org/D75390


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D75390





More information about the llvm-commits mailing list