Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GCC JIT build warnings on type width #55479

Closed
hoyosjs opened this issue Jul 11, 2021 · 1 comment
Closed

GCC JIT build warnings on type width #55479

hoyosjs opened this issue Jul 11, 2021 · 1 comment
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI untriaged New issue has not been triaged by the area owner

Comments

@hoyosjs
Copy link
Member

hoyosjs commented Jul 11, 2021

The GCC build of the JIT has plenty of warnings that look like:

/__w/1/s/src/coreclr/jit/emit.h:534:25: warning: 'emitter::emitAddrMode::amScale' is too small to hold all values of 'enum emitter::opSize'
  534 |         emitter::opSize amScale : 2;
      |                         ^~~~~~~
/__w/1/s/src/coreclr/jit/emit.h:853:27: warning: 'emitter::instrDesc::idAddrUnion::<unnamed struct>::_idReg3' is too small to hold all values of 'regNumber' {aka 'enum _regNumber_enum'}
  853 |                 regNumber _idReg3 : REGNUM_BITS;
      |                           ^~~~~~~
/__w/1/s/src/coreclr/jit/emit.h:854:27: warning: 'emitter::instrDesc::idAddrUnion::<unnamed struct>::_idReg4' is too small to hold all values of 'regNumber' {aka 'enum _regNumber_enum'}
  854 |                 regNumber _idReg4 : REGNUM_BITS;
      |                           ^~~~~~~
/__w/1/s/src/coreclr/jit/emit.h:667:19: warning: 'emitter::instrDesc::_idReg1' is too small to hold all values of 'regNumber' {aka 'enum _regNumber_enum'}
  667 |         regNumber _idReg1 : REGNUM_BITS; // register num
      |                   ^~~~~~~
/__w/1/s/src/coreclr/jit/emit.h:669:19: warning: 'emitter::instrDesc::_idReg2' is too small to hold all values of 'regNumber' {aka 'enum _regNumber_enum'}
  669 |         regNumber _idReg2 : REGNUM_BITS;
      |   

cc: @am11, @dotnet/jit-contrib

@dotnet-issue-labeler dotnet-issue-labeler bot added area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI untriaged New issue has not been triaged by the area owner labels Jul 11, 2021
@am11
Copy link
Member

am11 commented Jul 11, 2021

It is the duplicate of #33541. The original issue was fixed by updating gcc to v9, the remaining warnings are non-critical but correct depending on how developer or compiler look at it (#33541 (comment)).

@hoyosjs hoyosjs closed this as completed Jul 11, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Aug 11, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-CodeGen-coreclr CLR JIT compiler in src/coreclr/src/jit and related components such as SuperPMI untriaged New issue has not been triaged by the area owner
Projects
None yet
Development

No branches or pull requests

2 participants