[PATCH] D78203: [VP,Integer,#2] ExpandVectorPredication pass

Simon Moll via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 3 00:24:54 PDT 2021


simoll added a comment.

In D78203#2730888 <https://reviews.llvm.org/D78203#2730888>, @aprantl wrote:

> Since it's already late on Friday I took the liberty to revert the commit so the bots can recover and you can investigate the failure at your own pace. Let me know if I can help with the fix!

Thanks for taking action! I'll look into how `include/llvm/CodeGen/MachinePassRegistry.def` actually works.. i suppose moving the entry from `DUMMY_FUNCTION_PASS` to `FUNCTION_PASS` should fix this (will reproduce and fix locally). Never had a revert, what is the usual procedure to recommit after revert? (update/new diff, review, or just commit with the fix?)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78203



More information about the llvm-commits mailing list