[LinuxBIOS] LNXI Merge: lnxi-patch-11/16
Jason Schildt
jschildt at lnxi.com
Tue Sep 6 16:18:19 CEST 2005
On Tue, Sep 06, 2005 at 12:08:50PM +0200, Stefan Reinauer wrote:
> * jason schildt <jschildt at lnxi.com> [050903 00:03]:
> > +/* We can reduce the size of code generated by romcc by
> > + * changing all of the fixed size types that live in registers
> > + * into simple unsigned variables. (ie s/uint8_t/unsigned/g)
> > + */
>
> Why is this? I would consider specifying an 8bit type to be more
> space-safing than using some generic untyped integer value. If not this
> should be fixed in romcc..
You're right in your thinking, but at this time ROMCC converts it to an
unsigned anyway and winds up producing more code doing so. So in the
end the space saving measure costs more in code size produced by ROMCC.
>
> > /* AMD K8 Unsupported 1Ghz? */
> > if (id == (PCI_VENDOR_ID_AMD | (0x1100 << 16))) {
> > - if (is_cpu_pre_e0()) // CK804 support 1G?
>
> > + device_t dev_2 = PCI_DEV(0,0x18,2);
> > + if(pci_read_config32(dev_2,0x9c) < 0x20f00) {
>
> The function call looks a lot more readable here. How much is the gain
> of manual inlining here?
>
> Stefan
>
> --
> LinuxBIOS mailing list
> LinuxBIOS at openbios.org
> http://www.openbios.org/mailman/listinfo/linuxbios
--
Jason W. Schildt
LinuxBIOS Software Engineer
Linux Networx
More information about the coreboot
mailing list