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

2021-06-15 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG3302af9d4c39: Support: Remove F_{None,Text,Append} compatibility synonyms, NFC (authored by dexonsmith). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

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

2021-04-30 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith added a comment. In D101506#2729839 , @MaskRay wrote: > Perhaps give us and other projects one week or longer ... Sure thing; I'll try committing next week some time. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101506/new/

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

2021-04-30 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay accepted this revision. MaskRay added a comment. This revision is now accepted and ready to land. Perhaps give us and other projects one week or longer ... CHANGES SINCE LAST ACTION https://reviews.llvm.org/D101506/new/ https://reviews.llvm.org/D101506

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

2021-04-30 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith updated this revision to Diff 341966. dexonsmith edited the summary of this revision. dexonsmith added a comment. Rebased on top of https://reviews.llvm.org/D101506 -- that updates all in-tree users, and now this is just removing the API. CHANGES SINCE LAST ACTION

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

2021-04-29 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith added a comment. In 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.

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

2021-04-29 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. 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 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

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

2021-04-28 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. 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

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

2021-04-28 Thread Duncan P. N. Exon Smith via Phabricator via cfe-commits
dexonsmith created this revision. dexonsmith added a reviewer: zturner. Herald added subscribers: dcaballe, cota, teijeong, rdzhabarov, tatianashp, bmahjour, msifontes, jurahul, Kayjukh, grosul1, jvesely, Joonsoo, liufengdb, aartbik, lucyrfox, mgester, arpith-jacob, antiagainst, shauheen,