Skip to content

Commit

Permalink
Fix "anonymous types declared in an anonymous union" warnings
Browse files Browse the repository at this point in the history
They look like this:
```
    CC src/processor.o
In file included from /Users/mhorn/Projekte/Julia/julia.master/src/processor.cpp:10:
In file included from ./processor.h:5:
./julia.h:395:9: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
        struct {
        ^
./julia.h:405:9: warning: anonymous types declared in an anonymous union are an extension [-Wnested-anon-types]
        struct {
        ^
2 warnings generated.
```
and come from code that was introduced by @Keno in PR JuliaLang#43852.

But it turns out that the union is not used at all! So I'm simply removing
the offending union. Perhaps it is needed for some future work, but it should
be trivial to add it back if needed. If that happens, I suggest a comment
is added that explain why this looks similar to but has different layout
compared to the `typedef _jl_purity_overrides_t` also in `julia.h`.
  • Loading branch information
fingolfin committed Mar 31, 2022
1 parent 82ce311 commit 1c1c5eb
Showing 1 changed file with 2 additions and 20 deletions.
22 changes: 2 additions & 20 deletions src/julia.h
Original file line number Diff line number Diff line change
Expand Up @@ -390,26 +390,8 @@ typedef struct _jl_code_instance_t {
//TODO: uint8_t absolute_max; // whether true max world is unknown

// purity results
union {
uint32_t ipo_purity_bits;
struct {
uint8_t ipo_consistent:2;
uint8_t ipo_effect_free:2;
uint8_t ipo_nothrow:2;
uint8_t ipo_terminates:2;
uint8_t ipo_nonoverlayed:1;
} ipo_purity_flags;
};
union {
uint32_t purity_bits;
struct {
uint8_t consistent:2;
uint8_t effect_free:2;
uint8_t nothrow:2;
uint8_t terminates:2;
uint8_t nonoverlayed:1;
} purity_flags;
};
uint32_t ipo_purity_bits;
uint32_t purity_bits;
jl_value_t *argescapes; // escape information of call arguments

// compilation state cache
Expand Down

0 comments on commit 1c1c5eb

Please sign in to comment.