[PATCH] D101506: Support: Remove F_{None,Text,Append} compatibility synonyms, NFC

Duncan P. N. Exon Smith via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 29 19:24:36 PDT 2021


dexonsmith added a comment.

In D101506#2724746 <https://reviews.llvm.org/D101506#2724746>, @MaskRay wrote:

> Perhaps split the migration part from the `OpenFlags` removal part. I thought about deleting the aliases too but refrained because I noticed our internal code base needs migration.
> Since you propose this, I will start migration and hopefully can finish that next week so that you will not be blocked.

SGTM; I almost split it up from the beginning, not sure what made me lazy about this time. I'll try to post the split tomorrow.

In D101506#2727306 <https://reviews.llvm.org/D101506#2727306>, @MaskRay wrote:

> I've fixed Halide and sent patches for https://github.com/JuliaLang/julia/pull/40666 & https://github.com/rust-lang/rust/pull/84724 & https://github.com/GPUOpen-Drivers/llpc/pull/1232

Great! Thanks.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101506



More information about the cfe-commits mailing list