Skip to content

Conversation

varkor
Copy link
Contributor

@varkor varkor commented Oct 21, 2020

Closes #73298.

r? @lcnr

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 21, 2020
@varkor varkor added A-const-generics Area: const generics (parameters and arguments) F-const_generics `#![feature(const_generics)]` labels Oct 21, 2020
@lcnr
Copy link
Contributor

lcnr commented Oct 21, 2020

I guess we previously didn't normalize/evaluate that constant in symbol mangling?

r=me once the nit is fixed

@varkor
Copy link
Contributor Author

varkor commented Oct 21, 2020

I guess we previously didn't normalize/evaluate that constant in symbol mangling?

An issue with symbol mangling was my guess, but I didn't investigate.

@lcnr
Copy link
Contributor

lcnr commented Oct 22, 2020

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Oct 22, 2020

📌 Commit 10d6f2d has been approved by lcnr

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Oct 22, 2020
Dylan-DPC-zz pushed a commit to Dylan-DPC-zz/rust that referenced this pull request Oct 22, 2020
@Dylan-DPC-zz
Copy link

failed in rollup

@bors r-

@bors bors added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 22, 2020
@varkor
Copy link
Contributor Author

varkor commented Oct 22, 2020

Interesting. We hadn't encountered that error before.

@lcnr
Copy link
Contributor

lcnr commented Oct 26, 2020

I failed to reproduce this 🤔

let's try again, maybe it was spurious (even if that would be weird)

@bors r+ rollup=never

@bors
Copy link
Collaborator

bors commented Oct 26, 2020

📌 Commit 10d6f2d has been approved by lcnr

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Oct 26, 2020
@bors
Copy link
Collaborator

bors commented Oct 26, 2020

⌛ Testing commit 10d6f2d with merge d4cdcd7fa98c51e41cdcd7876b5624cb8fe9f5fd...

@bors
Copy link
Collaborator

bors commented Oct 26, 2020

💔 Test failed - checks-actions

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Oct 26, 2020
@varkor
Copy link
Contributor Author

varkor commented Oct 28, 2020

I failed to reproduce this 🤔

Me too. Curious. Let's close this for now and add a comment to the original issue.

@rust-log-analyzer
Copy link
Collaborator

The job x86_64-gnu-llvm-12 failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)
........................................................................................ 1848/13121
............................................................................i........... 1936/13121
........................................................................................ 2024/13121
........................................................................................ 2112/13121
.......................................F............F................................... 2200/13121
........................................................................................ 2376/13121
........................................................................................ 2464/13121
........................................................................................ 2552/13121
........................................................................................ 2640/13121
---
failures:

---- [ui] src/test/ui/const-generics/issue-73298.rs#full stdout ----

error in revision `full`: test compilation failed although it shouldn't!
status: exit status: 1
command: "/checkout/obj/build/x86_64-unknown-linux-gnu/stage2/bin/rustc" "/checkout/src/test/ui/const-generics/issue-73298.rs" "-Zthreads=1" "--target=x86_64-unknown-linux-gnu" "--cfg" "full" "--error-format" "json" "--json" "future-incompat" "-Ccodegen-units=1" "-Zui-testing" "-Zdeduplicate-diagnostics=no" "-Cstrip=debuginfo" "-C" "prefer-dynamic" "--out-dir" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/const-generics/issue-73298.full" "-A" "unused" "-Crpath" "-O" "-Cdebuginfo=0" "-Lnative=/checkout/obj/build/x86_64-unknown-linux-gnu/native/rust-test-helpers" "-L" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/const-generics/issue-73298.full/auxiliary"
stdout: none
--- stderr -------------------------------
  --> /checkout/src/test/ui/const-generics/issue-73298.rs:4:27
   |
   |
LL | #![cfg_attr(full, feature(const_generics))]
   |
   |
   = note: removed in favor of `#![feature(adt_const_params)]` and `#![feature(generic_const_exprs)]`
error: aborting due to previous error

For more information about this error, try `rustc --explain E0557`.
------------------------------------------
---
warning: the feature `min_const_generics` has been stable since 1.51.0 and no longer requires an attribute to enable
Some tests failed in compiletest suite=ui mode=ui host=x86_64-unknown-linux-gnu target=x86_64-unknown-linux-gnu
  --> $DIR/issue-73298.rs:6:26
   |
LL | #![cfg_attr(min, feature(min_const_generics))]
   |
   = note: `#[warn(stable_features)]` on by default

warning: 1 warning emitted
warning: 1 warning emitted




The actual stderr differed from the expected stderr.
Actual stderr saved to /checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/const-generics/issue-73298.min/issue-73298.min.stderr
To only update this specific test, also pass `--test-args const-generics/issue-73298.rs`


error in revision `min`: 1 errors occurred comparing output.
status: exit status: 0
command: "/checkout/obj/build/x86_64-unknown-linux-gnu/stage2/bin/rustc" "/checkout/src/test/ui/const-generics/issue-73298.rs" "-Zthreads=1" "--target=x86_64-unknown-linux-gnu" "--cfg" "min" "--error-format" "json" "--json" "future-incompat" "-Ccodegen-units=1" "-Zui-testing" "-Zdeduplicate-diagnostics=no" "-Cstrip=debuginfo" "-C" "prefer-dynamic" "--out-dir" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/const-generics/issue-73298.min" "-A" "unused" "-Crpath" "-O" "-Cdebuginfo=0" "-Lnative=/checkout/obj/build/x86_64-unknown-linux-gnu/native/rust-test-helpers" "-L" "/checkout/obj/build/x86_64-unknown-linux-gnu/test/ui/const-generics/issue-73298.min/auxiliary"
stdout: none
--- stderr -------------------------------
warning: the feature `min_const_generics` has been stable since 1.51.0 and no longer requires an attribute to enable
   |
   |
LL | #![cfg_attr(min, feature(min_const_generics))]
   |
   = note: `#[warn(stable_features)]` on by default

warning: 1 warning emitted

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-const-generics Area: const generics (parameters and arguments) F-const_generics `#![feature(const_generics)]` S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"error: linking with cc failed: exit code: 1" regression in release build with const generics
6 participants