[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