HomePhabricator

[MCA] Fix wrong definition of ResourceUnitMask in DefaultResourceStrategy.
Audit RequiredrL350820

Description

[MCA] Fix wrong definition of ResourceUnitMask in DefaultResourceStrategy.

Field ResourceUnitMask was incorrectly defined as a 'const unsigned' mask. It
should have been a 64 bit quantity instead. That means, ResourceUnitMask was
always implicitly truncated to a 32 bit quantity.
This issue has been found by inspection. Surprisingly, that bug was latent, and
it never negatively affected any existing upstream targets.

This patch fixes the wrong definition of ResourceUnitMask, and adds a bunch of
extra debug prints to help debugging potential issues related to invalid
processor resource masks.

Details

Auditors
grosbach
Committed
adibiagioJan 10 2019, 5:59 AM
Parents
rL350819: [LLD][ELF] - Support MSP430.
Branches
Unknown
Tags
Unknown