craig.topper added inline comments.
================
Comment at: clang/include/clang/Basic/riscv_vector.td:651
+ BasicBlock *IsNotNull = createBasicBlock("newvl.isNotNull",
this->CurFn);
+ BasicBlock *IsNull = createBasicBlock("newvl.isNull", this->CurFn);
+ Builder.CreateCondBr(Cmp, IsNotNull, IsNull);
----------------
craig.topper wrote:
> newvl.isNull doesn't make sense since isNotNull eventually jumps to it.
>
> Maybe use "store_newvl" and "store_newvl_end"?
I changed my mind. How about "newvl_store" and "newvl_end"?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D126461/new/
https://reviews.llvm.org/D126461
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
