Skip to content

Commit

Permalink
Merge pull request ocaml#12914 from dustanddreams/clang_s390x
Browse files Browse the repository at this point in the history
s390x: allow builds with llvm/clang
  • Loading branch information
gasche authored Jan 18, 2024
2 parents 8fa4c5f + c3494a2 commit 42ece63
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 2 deletions.
4 changes: 4 additions & 0 deletions Changes
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,10 @@ _______________
bytecode linker in `-custom` and `-output-*` modes.
(David Allsopp, Antonin Décimo and Samuel Hym, review by Vincent Laviron)

- #xxxxx: Slightly change the s390x assembly dialect in order to build with
Clang's integrated assembler.
(Miod Vallat, review by TBD)

### Build system:

- #12909: Reorganise how MKEXE_VIA_CC is built to make it correct for MSVC by
Expand Down
2 changes: 1 addition & 1 deletion asmcomp/s390x/emit.mlp
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ let emit_stack env r =

let emit_load_symbol_addr reg s =
if !pic_code then
` lgrl {emit_reg reg}, {emit_symbol s}@GOTENT\n`
` lgrl {emit_reg reg}, {emit_symbol s}@GOT\n`
else
` larl {emit_reg reg}, {emit_symbol s}\n`

Expand Down
2 changes: 1 addition & 1 deletion runtime/s390x.S
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@

#define LBL(x) .L##x
#define G(r) r
#define GREL(r) r@GOTENT
#define GREL(r) r@GOT
#define GCALL(r) r@PLT
#define TEXT_SECTION(name)
#define FUNCTION(name) \
Expand Down

0 comments on commit 42ece63

Please sign in to comment.