[PATCH] D62773: [llvm] [test] Remove non-portable EISDIR test from macho-disassemble-g-dsym.test

Michał Górny via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 1 05:15:25 PDT 2019


mgorny created this revision.
mgorny added reviewers: mtrent, pete, lhames, JDevlieghere, krytarowski.
Herald added a subscriber: rupprecht.
Herald added a project: LLVM.

Remove the test checking error message for 'is a directory'.  It does
not seem to serve any real purpose, and it relies on matching platform
error strings which are unpredictable and makes the test fragile.
Furthermore, it fails on NetBSD where read() works on directories,
and therefore does not return EISDIR at all.

Fixes r362141.


Repository:
  rL LLVM

https://reviews.llvm.org/D62773

Files:
  llvm/test/tools/llvm-objdump/X86/macho-disassemble-g-dsym.test


Index: llvm/test/tools/llvm-objdump/X86/macho-disassemble-g-dsym.test
===================================================================
--- llvm/test/tools/llvm-objdump/X86/macho-disassemble-g-dsym.test
+++ llvm/test/tools/llvm-objdump/X86/macho-disassemble-g-dsym.test
@@ -15,8 +15,3 @@
 // RUN: llvm-objdump -m -d -g -dsym %p/../Inputs/libbogus11.a %p/../../dsymutil/Inputs/basic.macho.x86_64 2>&1 | FileCheck -check-prefix BAD_INPUT %s
 
 BAD_INPUT: is not a Mach-O or Universal file type.
-
-// RUN: not llvm-objdump -m -d -g -dsym %p/Inputs %p/Inputs/hello-macho-thin 2>&1 | FileCheck -check-prefix DIRECTORY %s
-
-// Windows will emit "Is a directory", whereas others emit "is a directory"
-DIRECTORY: {{[i|I]}}s a directory


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D62773.202544.patch
Type: text/x-patch
Size: 733 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190601/30202725/attachment.bin>


More information about the llvm-commits mailing list