[PATCH] D79578: [mlir] NFC: Rename LoopOps dialect to SCF (Structured Control Flow)
Pete Steinfeld via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 11 11:18:07 PDT 2020
PeteSteinfeld added a comment.
> I don't think this is the request. The request is just "don't break the build".
> The pre-merge testing will test flang as well, but you would have had to rebase the patch after you landed the dependent ones first so that the pre-merge testing could work (I think marking the dependencies in Phabricator would have worked also).
>
> In general trying to get the pre-merge testing green before pushing changes is safer.
Yes, the request is to not break the build. And in the case where the pre-merge testing is not working, I encourage you to contact @schweitz or @PeteSteinfeld or any other member of the flang team to verify that our builds succeed.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D79578/new/
https://reviews.llvm.org/D79578
More information about the llvm-commits
mailing list