[PATCH] D121898: [Verifier] Verify parameter alignment.

Phoebe Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 21 23:14:12 PDT 2022


pengfei accepted this revision.
pengfei added a comment.

Still fine to me.



================
Comment at: llvm/test/Verifier/param-ret-align.ll:1
+; RUN: not llvm-as < %s 2>&1 | FileCheck %s
+
----------------
skan wrote:
> LuoYuanke wrote:
> > pengfei wrote:
> > > Can we merge them to one file?
> > Probably not. llvm-as would exit on the first failure.
> Then? We could check more than one failure in a LIT test.
I understand it is the second function will never be handled once the first failed. With a rough search, I didn't find an example to handle more than one fucntions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121898



More information about the llvm-commits mailing list