[PATCH] D100981: Delete le32/le64 targets

MaskRay added a comment. In D100981#2709450 , @thakis wrote: > maskray: your undelete didn't add back BuiltinsLe64.def so builds are broken > atm Fixed by vitalybuka Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.ll

[PATCH] D100981: Delete le32/le64 targets

thakis added a comment. maskray: your undelete didn't add back BuiltinsLe64.def so builds are broken atm Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D100981/new/ https://reviews.llvm.org/D100981 ___ cfe

[PATCH] D100981: Delete le32/le64 targets

dschuff added a comment. Would it make sense for you to to upstream an LLVM target such as le32-halide? (Or perhaps even arm32-halide or some other?) Then you'd actually have more control over your own codegen, datalayout, etc. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D100981: Delete le32/le64 targets

srj added a comment. In D100981#2709205 , @MaskRay wrote: > In D100981#2709044 , @srj wrote: > >> Any chance that this could be (temporarily) reverted? This will break >> literally all Halide compilation; we're w

[PATCH] D100981: Delete le32/le64 targets

MaskRay added a comment. Herald added a subscriber: tmatheson. In D100981#2709044 , @srj wrote: > Any chance that this could be (temporarily) reverted? This will break > literally all Halide compilation; we're working on a fix on our side, but it > woul

[PATCH] D100981: Delete le32/le64 targets

srj added a comment. Any chance that this could be (temporarily) reverted? This will break literally all Halide compilation; we're working on a fix on our side, but it would be nice to have a few days to be sure we have it right, and I suspect there's no urgency to removing this right now. Re

[PATCH] D100981: Delete le32/le64 targets

abadams added a comment. In D100981#2706899 , @dschuff wrote: > Thanks. I had heard in the past that there were some other folks who had used > le32/le64 as a "generic" target (in fact that's why it's named so > generically, rather than being called "pn

[PATCH] D100981: Delete le32/le64 targets

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG77ac823fd285: Delete le32/le64 targets (authored by MaskRay). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm

[PATCH] D100981: Delete le32/le64 targets

dschuff accepted this revision. dschuff added a comment. This revision is now accepted and ready to land. Thanks. I had heard in the past that there were some other folks who had used le32/le64 as a "generic" target (in fact that's why it's named so generically, rather than being called "pnacl"

[PATCH] D100981: Delete le32/le64 targets

MaskRay created this revision. MaskRay added a reviewer: dschuff. Herald added subscribers: dexonsmith, jfb, hiraditya, mgorny. MaskRay requested review of this revision. Herald added subscribers: llvm-commits, cfe-commits, aheejin. Herald added projects: clang, LLVM. They are unused now. Note: N