fix usage of cfg_attr

Elide the functions completely when miri or NO_UI_TESTS are enabled.

Extracted from a patch by Benno Lossin.

Signed-off-by: Paolo Bonzini <[email protected]>

Author: bonzini

tests/ui.rs

@@ -1,4 +1,5 @@
-#[cfg_attr(not(any(miri, NO_UI_TESTS)), test)]
+#[cfg(not(any(miri, NO_UI_TESTS)))]
+#[test]
 fn compile_fail() {
     let test_cases = trybuild::TestCases::new();
     test_cases.compile_fail("tests/ui/compile-fail/pinned_drop/*.rs");
@@ -7,7 +8,8 @@ fn compile_fail() {
     test_cases.compile_fail("tests/ui/compile-fail/zeroable/*.rs");
 }
 
-#[cfg_attr(not(any(miri, NO_UI_TESTS)), test)]
+#[cfg(not(any(miri, NO_UI_TESTS)))]
+#[test]
 fn expand() {
     macrotest::expand("tests/ui/expand/*.rs");
 }