[PATCH] D32652: Gracefully handle empty .drectve sections

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 1 22:18:12 PDT 2017


ruiu added inline comments.


================
Comment at: test/Object/coff-empty-drectve.test:3
+
+CHECK: Directive(s):
+
----------------
You want to add `{{$}}` at end of this line to make sure that `Directive(s):` is not followed by any other characters.


================
Comment at: test/Object/coff-empty-drectve.test:4
+CHECK: Directive(s):
+
----------------
Remove trailing empty line.


https://reviews.llvm.org/D32652





More information about the llvm-commits mailing list