Skip to content

PR #606 Build simple ROMs for CI testing of APL etc breaks compilation with xcc #620

Closed

Description

@lbetlej

Looks like PR #606 breaks compilation of UP & SMP targets (APL, CNL etc.) with xcc.
See compilation logs of failed Internal Intel CI at https://quickbuild.igk.intel.com/build/3259834

Activity

  1. added
    bugSomething isn't working as expected
    APLApplies to Apollolake platform
    CNLApplies to Cannonlake platform
    GLKApplies to Gemini Lake platform
    on Nov 23, 2018
  2. jajanusz commented on Nov 23, 2018

    @jajanusz
    Contributor

    It looks like gcc has some defaults that xcc is missing, idk, something like that makes it build for APL:
    jajanusz@d072209
    but I'm not sure that it should be like that.

    @lgirdwood
    I guess that will be needed only for qemu+gcc environment, so if there is no quickfix for xcc I can add flag like --enable-roms, so you will build it only for your CIs and we won't when we don't need it.

  3. lgirdwood commented on Nov 23, 2018

    @lgirdwood
    Member

    @jajanusz yes thanks, only needed for qemu CI boot test for each PRs, so not needed for other reasons.

  4. jajanusz commented on Nov 23, 2018

    @jajanusz
    Contributor

    With #621 ROMs are build only when --enable-roms is passed to ./confgiure script

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

APLApplies to Apollolake platformCNLApplies to Cannonlake platformGLKApplies to Gemini Lake platformbugSomething isn't working as expectedverifiedQA/Bug reporter verifies solution worked.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions