[PATCH] D88859: APINotes: add APINotesYAMLCompiler

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 5 11:58:07 PST 2020


thakis added a comment.

Looks like the test doesn't pass on Windows: http://45.33.8.238/win/27342/step_7.txt

Please take a look, and revert if it takes a while to fix.

Here's the output of the diff on that Win bot:

  thakis at thakis6-w MINGW64 /c/src/llvm-project (merge)
  $ diff --strip-trailing-cr out/gn/obj/clang/test/APINotes/Output/yaml-roundtrip.test.tmp.result clang/test/APINotes/Inputs/Frameworks/Simple.framework/Headers/Simple.apinotes
  1d0
  < ---
  8c7
  <         Nullability:     Nonnull
  ---
  >         Nullability:     N
  14c13
  <         Nullability:     Optional
  ---
  >         Nullability:     O
  20c19
  <         Nullability:     Unspecified
  ---
  >         Nullability:     U
  26c25
  <         Nullability:     Unspecified
  ---
  >         Nullability:     S
  29,30c28
  <         Nullability:     Unspecified
  < ...
  ---
  >         Nullability:     Scalar


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88859



More information about the cfe-commits mailing list