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

cgen: fix regression with .nodecl consts #785

Merged
merged 1 commit into from
Jul 7, 2023

Commits on Jul 7, 2023

  1. cgen: fix regression with .nodecl consts

    Summary
    =======
    
    Fix a regression where the `.nodecl` pragma was ignored by the C code
    generator in the context of constants.
    
    Details
    =======
    
    Apart from the fix, the single-use `genConstSetup` is inlined into
    `useConst`. Since the loc for constants is setup in `genConstDefinition`
    (which is always called before `useConst`), the `fillLoc` call is not
    added to `useConst` (this is also a step towards `fillLoc` only being
    used in procedures responsible for definitions).
    
    In addition, a test for the current behaviour of imported `.nodecl`
    constants is added.
    zerbina committed Jul 7, 2023
    Configuration menu
    Copy the full SHA
    53d601b View commit details
    Browse the repository at this point in the history