rust/crates
bors[bot] 6e10a9f578
Merge #2479
2479: Add expansion infrastructure for derive macros r=matklad a=flodiebold

I thought I'd experiment a bit with attribute macro/derive expansion, and here's what I've got so far. It has dummy implementations of the Copy / Clone derives, to show that the approach works; it doesn't add any attribute macro support, but I think that fits into the architecture.

Basically, during raw item collection, we look at the attributes and generate macro calls for them if necessary. Currently I only do this for derives, and just add the derive macro calls as separate calls next to the item. I think for derives, it's important that they don't obscure the actual item, since they can't actually change it (e.g. sending the item token tree through macro expansion unnecessarily might make completion within it more complicated).

Attribute macros would have to be recognized at that stage and replace the item (i.e., the raw item collector will just emit an attribute macro call, and not the item). I think when we implement this, we should try to recognize known inert attributes, so that we don't do macro expansion unnecessarily; anything that isn't known needs to be treated as a possible attribute macro call (since the raw item collector can't resolve the macro yet).

There's basically no name resolution for attribute macros implemented, I just hardcoded the built-in derives. In the future, the built-ins should work within the normal name resolution infrastructure; the problem there is that the builtin stubs in `std` use macros 2.0, which we don't support yet (and adding support is outside the scope of this).

One aspect that I don't really have a solution for, but I don't know how important it is, is removing the attribute itself from its input. I'm pretty sure rustc leaves out the attribute macro from the input, but to do that, we'd have to create a completely new syntax node. I guess we could do it when / after converting to a token tree.

Co-authored-by: Florian Diebold <flodiebold@gmail.com>
2019-12-05 20:00:20 +00:00
..
ra_arena Switch to variant-granularity field type inference 2019-11-25 00:12:36 +03:00
ra_assists Merge #2018 2019-12-02 15:52:11 +00:00
ra_batch rename ra_ide_api -> ra_ide 2019-11-27 21:35:06 +03:00
ra_cfg Disable doctests 2019-11-17 18:35:05 +03:00
ra_cli Remove log dependency from ra_cli 2019-11-29 19:40:59 -05:00
ra_db Run rustfmt with respect to Cargo.toml edition 2019-12-04 23:05:01 +01:00
ra_fmt fix 2190; add test for "replace if let with match" 2019-11-20 19:01:06 +01:00
ra_hir Merge #2479 2019-12-05 20:00:20 +00:00
ra_hir_def Merge #2479 2019-12-05 20:00:20 +00:00
ra_hir_expand Implement all the other built-in derives 2019-12-05 19:52:52 +01:00
ra_hir_ty Implement derive(Copy, Clone) properly (well, kind of) 2019-12-05 19:29:57 +01:00
ra_ide One pub function less is good! 2019-12-05 08:28:31 -05:00
ra_lsp_server Run rustfmt with respect to Cargo.toml edition 2019-12-04 23:05:01 +01:00
ra_mbe Don't wrap most syntax trees in invisible delimiters when converting to token tree 2019-12-05 19:29:39 +01:00
ra_parser Fix hir for ast::UnionDef 2019-11-25 17:50:49 +03:00
ra_prof Disable doctests 2019-11-17 18:35:05 +03:00
ra_project_model Merge #2381 2019-11-24 15:59:47 +00:00
ra_syntax ⬆️ rowan 2019-12-04 17:15:55 +01:00
ra_text_edit Disable doctests 2019-11-17 18:35:05 +03:00
ra_tt Derive Hash for TokenTrees 2019-11-28 13:41:58 +08:00
ra_vfs_glob Disable doctests 2019-11-17 18:35:05 +03:00
test_utils Disable doctests 2019-11-17 18:35:05 +03:00