Skip to content

Remove old x86 directives #5

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

Open
wants to merge 23 commits into
base: new-directives-in-x86
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
db8aba1
removed unary negation from flambda (#3572)
jvanburen Apr 25, 2025
de413fd
Avoid needless dependency on parameter .cmi by introducing `Parameter…
lukemaurer Apr 29, 2025
36fe15d
Reorganize docs (#3935)
ccasin Apr 29, 2025
c86a597
Reverse Poptrap instructions before Return (#3924)
gretay-js Apr 29, 2025
edcc229
"module type of" zap modalities to floor (#3922)
riaqn Apr 30, 2025
e8058d5
Add replay history to match names between two downwards traversals (#…
Gbury Apr 30, 2025
d95e9c9
Add doc pages for SIMD/small-numbers (#3936)
TheNumbat Apr 30, 2025
7f1f234
Remove unused function (#3950)
gretay-js May 1, 2025
b04e920
Store instruction id sequence in CFG (#3933)
xclerc May 1, 2025
e415e96
Refactor test for simd arrays (#3958)
gretay-js May 2, 2025
de0296d
Format tests (#3957)
gretay-js May 2, 2025
83590eb
Remove duplicate VEX prefix (#3963)
TheNumbat May 2, 2025
6e3762a
Configurable fiber stack sizes (#3773)
TheNumbat May 6, 2025
768b131
Fix "regalloc" profile category (#3968)
xclerc May 6, 2025
9994ecd
Fix miscompilation / liveness errors for string operations (#3960)
mshinwell May 6, 2025
be6e477
Format bytecomp/bytegen.ml (#3953)
jvanburen May 6, 2025
58b8e0c
Add Blambda IR for bytecode compilation (#3590)
jvanburen May 6, 2025
957e5ce
git blame ignore reformat bytegen/bytecomp.ml (#3971)
jvanburen May 6, 2025
54d6467
Fix incorrect sort in `type_argument` (#3972)
ccasin May 6, 2025
66a06b9
Use new directives on x86 (#3931)
spiessimon May 8, 2025
7281f3d
remove old directives
spiessimon Apr 25, 2025
c73e03b
update binary emitter filling code
spiessimon May 8, 2025
dd85f4f
review
spiessimon May 8, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
3 changes: 3 additions & 0 deletions .git-blame-ignore-revs
Original file line number Diff line number Diff line change
Expand Up @@ -6,3 +6,6 @@

# Reformat to cope with .ocamlformat change
9777982cfb3733cd21896797b0b21980f5473462

# Reformat bytegen/bytecomp.ml (#3953)
be6e477ef3177e765c665752df7bf16ae1865ed9
62 changes: 41 additions & 21 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -178,7 +178,6 @@ jobs:
env:
J: "3"
run_testsuite: "true"
expected_fail: "${{matrix.expected_fail == true}}"

steps:
- name: Checkout the Flambda backend repo
Expand All @@ -200,12 +199,14 @@ jobs:
if: matrix.os == 'macos-latest'
run: HOMEBREW_NO_INSTALL_CLEANUP=TRUE brew install autoconf

# NB. The "rev" number in the cache key below must be updated each time
# the patch file is changed!
- name: Cache OCaml 4.14, dune and menhir
uses: actions/cache@v4
id: cache
with:
path: ${{ github.workspace }}/ocaml-414/_install
key: ${{ matrix.os }}-cache-ocaml-414-patched-dune-3152-menhir-20231231
key: ${{ matrix.os }}-cache-ocaml-414-patched-dune-3152-menhir-20231231-rev13

- name: Checkout OCaml 4.14
uses: actions/checkout@master
Expand Down Expand Up @@ -285,14 +286,23 @@ jobs:
--with-dune=$GITHUB_WORKSPACE/ocaml-414/_install/bin/dune \
${{ matrix.config }}

- name: Setup for saving core files (not for macOS at the moment)
- name: Setup for saving core files (not for macOS)
if: matrix.os != 'macos-latest'
run: |
sudo mkdir /cores
sudo chmod 777 /cores
# Core filenames will be of the form executable.pid.timestamp:
sudo bash -c 'echo "/cores/%e.%p.%t" > /proc/sys/kernel/core_pattern'

- name: Setup for saving core files (macOS)
if: matrix.os == 'macos-latest'
run: |
sudo chmod 1777 /cores
sudo sysctl kern.coredump=1
/usr/libexec/PlistBuddy -c "Add :com.apple.security.get-task-allow bool true" /tmp/core.entitlements
codesign -s - -f --entitlements /tmp/core.entitlements $GITHUB_WORKSPACE/ocaml-414/_install/bin/ocamlc.opt
codesign -s - -f --entitlements /tmp/core.entitlements $GITHUB_WORKSPACE/ocaml-414/_install/bin/ocamlopt.opt

- name: Build, install and test Flambda backend
working-directory: flambda_backend
run: |
Expand All @@ -313,25 +323,35 @@ jobs:
run: |
PATH=$GITHUB_WORKSPACE/ocaml-414/_install/bin:$PATH make check_all_arches

# CR-soon xclerc for xclerc: re-enable the "upload-artifact" action
# (they are failing @4 because the names are not unique)
# - uses: actions/upload-artifact@v4
# if: ${{ failure() }} && matrix.os != 'macos-latest'
# with:
# name: cores-${{ github.sha }}
# path: /cores
#
# - uses: actions/upload-artifact@v4
# if: ${{ failure() }} && matrix.os != 'macos-latest'
# with:
# name: _build-${{ github.sha }}
# path: $GITHUB_WORKSPACE/_build
- uses: actions/upload-artifact@v4
if: ${{ failure() }}
with:
name: cores-${{ github.sha }}-${{ github.run_id }}-${{ matrix.name }}
path: /cores

# - uses: actions/upload-artifact@v4
# if: ${{ failure() }}
# with:
# name: ocaml-414-${{ github.sha }}-${{ github.run_id }}-${{ matrix.name }}
# path: ${{ github.workspace }}/ocaml-414/_install

- uses: actions/upload-artifact@v4
if: ${{ failure() }} && matrix.os == 'macos-latest'
with:
name: DiagnosticReports-${{ github.sha }}-${{ github.run_id }}-${{ matrix.name }}
path: /Users/runner/Library/Logs/DiagnosticReports

# - uses: actions/upload-artifact@v4
# if: ${{ failure() }}
# with:
# name: _build-${{ github.sha }}-${{ github.run_id }}-${{ matrix.name }}
# path: ${{ github.workspace }}/flambda_backend/_build
#
# - uses: actions/upload-artifact@v4
# if: ${{ failure() }} && matrix.os != 'macos-latest'
# with:
# name: _runtest-${{ github.sha }}
# path: $GITHUB_WORKSPACE/_runtest
# - uses: actions/upload-artifact@v4
# if: ${{ failure() }}
# with:
# name: _runtest-${{ github.sha }}-${{ github.run_id }}-${{ matrix.name }}
# path: ${{ github.workspace }}/flambda_backend/_runtest

concurrency:
group: ${{ github.workflow }}-${{ github.head_ref || github.run_id }}
Expand Down
Loading