[PATCH] D112899: Update fir.insert_on_range syntax to make the range more explicit (NFC)

Valentin Clement via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 22 13:06:41 PST 2021


clementval added a comment.

In D112899#3105919 <https://reviews.llvm.org/D112899#3105919>, @clementval wrote:

> Thanks Mehdi for this patch. We welcome changes that improve the readability of the code. Could you consider delaying these kind patches till we have finished upstreaming most of the changes, so as to avoid additional effort to merge this back into fir-dev.
> Kiran as volunteered to merge this back to fir-dev now.

@mehdi_amini This comment was mainly for any further syntax change in the dialect. This one can go in anytime.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112899



More information about the llvm-commits mailing list