[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
Mon Mar 2 12:55:33 PST 2020


clayborg added a comment.

Trying again with:

commit 57688350adea307e7bccb83b68a5b7333de31fd7 <https://reviews.llvm.org/rG57688350adea307e7bccb83b68a5b7333de31fd7> (HEAD -> master, origin/master, origin/HEAD)
Author: Greg Clayton <gclayton at fb.com>
Date:   Mon Mar 2 12:40:46 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. Also fixed an issue where strings for files in the file table could be added in opposite order due to parameters to function calls not having a strong ordering, which caused tests to fail.


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