Skip to content

Commit

Permalink
Fix a case of -Wmisleading-indentation when building with Clang (Ju…
Browse files Browse the repository at this point in the history
…liaLang#43472)

```
In file included from /usr/home/alex/julia/src/codegen.cpp:2005:
In file included from ./intrinsics.cpp:7:
In file included from ./ccall.cpp:322:
./abi_x86.cpp:78:9: warning: misleading indentation; statement is not part of the previous 'if' [-Wmisleading-indentation]
        ab.addByValAttr(Ty);
        ^
./abi_x86.cpp:73:5: note: previous statement is here
    if (is_complex64(dt) || is_complex128(dt) || (jl_is_primitivetype(dt) && size <= 8))
    ^
1 warning generated.
```
  • Loading branch information
ararslan committed Dec 18, 2021
1 parent 6817551 commit e68c047
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/abi_x86.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -73,9 +73,9 @@ bool needPassByRef(jl_datatype_t *dt, AttrBuilder &ab, LLVMContext &ctx, Type *T
if (is_complex64(dt) || is_complex128(dt) || (jl_is_primitivetype(dt) && size <= 8))
return false;
#if JL_LLVM_VERSION < 120000
ab.addAttribute(Attribute::ByVal);
ab.addAttribute(Attribute::ByVal);
#else
ab.addByValAttr(Ty);
ab.addByValAttr(Ty);
#endif
return true;
}
Expand Down

0 comments on commit e68c047

Please sign in to comment.