`rustc_data_structures::static_assert!(false)` does not error

[memoryruins]

1.40.0-nightly 2019-10-09 20cc75272619cc452e3a

The following does not error locally / on the playground https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=9dde03338a31e2f40491a66dc3a18761

#![feature(rustc_private)]

extern crate rustc_data_structures;
use rustc_data_structures::static_assert;

fn main() {
    // doesn't error, but should
    static_assert!(2 + 2 == 5);

    // correctly errors when uncommented
    // const _: () = [()][!(2 + 2 == 5) as usize];

    // https://doc.rust-lang.org/nightly/nightly-rustc/src/rustc_data_structures/macros.rs.html#4-11
}

If I copy paste the macro into the same file, it errors as expected https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=bec79345cfe7cecf1b92eed6c2736152

error: any use of this value will cause an error
  --> src/main.rs:11:23
   |
11 |         const _: () = [()][!($test: bool) as usize];
   |         --------------^^^^^^^^^^^^^^^^^^^^^^^^^^^^^-
   |                       |
   |                       index out of bounds: the len is 1 but the index is 1
...
16 |     static_assert!(2 + 2 == 5);
   |     --------------------------- in this macro invocation
   |
   = note: `#[deny(const_err)]` on by default

Checking other macros, rustc_data_structures::static_assert_size (which is defined next to static_assert) and static_assertions::const_assert error as expected.

[rodrimati1992]

This feels like it's related to a similar issue (#61058) I created a while back.

[Centril]

cc @oli-obk

[oli-obk]

This is probably because the "error" is just a lint. And we appear to silence it in external macros. We shouldn't do that. IIRC this is just a flag that needs to be set for the CONST_ERR lint macro

[Mark-Simulacrum]

Yes, we need to set report_in_external_macro param on the lint I believe

[memoryruins]

Ah, that makes sense! Building locally with the parameter set to confirm.
@rodrimati1992 it does ^^ will check the samples in there too.