[PATCH] D92807: [test] Rewrite phi-empty.ll into a unittest

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 8 08:09:39 PST 2020


spatel accepted this revision.
spatel added a comment.
This revision is now accepted and ready to land.

LGTM. 
Do you recommend that we explicitly test NPM when adding test files? I was not sure if we want that or just wait for the defaults to change to avoid the back-and-forth.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D92807



More information about the llvm-commits mailing list