Rust: Add an error for inline assembly which are non-volatile without any outputs

Created on 16 Nov 2017  路  2Comments  路  Source: rust-lang/rust

https://github.com/rust-lang/rust/pull/45920 turned up some usages of inline assembly which have no outputs. LLVM will just remove such inline assembly as they can never be used. We should make this an error.

A-inline-assembly C-feature-request
Source
picture Zoxc
❤1

Most helpful comment

It's worth considering an error though; inline asm is uncommon and extraordinarily unsafe, conditions which generally favor explicitness over implicitness.

picture sunfishcode on 16 Nov 2017
👍3

All 2 comments

Alternatively, do what GCC does and add volatile automatically if there are no outputs.

parched picture parched on 16 Nov 2017
👍1

It's worth considering an error though; inline asm is uncommon and extraordinarily unsafe, conditions which generally favor explicitness over implicitness.

sunfishcode picture sunfishcode on 16 Nov 2017
👍3
Was this page helpful?
0 / 5 - 0 ratings

Related issues

wthrowe picture wthrowe  路  3Comments
pedrohjordao picture pedrohjordao  路  3Comments
behnam picture behnam  路  3Comments
mcarton picture mcarton  路  3Comments
overvenus picture overvenus  路  3Comments