David Mak derppening
  • Joined on 2023-08-31
derppening commented on pull request M-Labs/nac3#319 2023-09-15 09:58:25 +08:00
Use LLVM New Pass Manager

Mostly to diagnose problems that occur only using the Legacy PM, e.g. #315.

derppening commented on pull request M-Labs/nac3#319 2023-09-14 15:38:29 +08:00
Use LLVM New Pass Manager

v4: Changed implementation to keep both legacy and new PMs for testing, refactored codegen test cases to test both legacy and new PMs.

Ready for review.

derppening pushed to llvm-new-pass-manager at M-Labs/nac3 2023-09-14 15:36:32 +08:00
667ebc699c meta: Allow specifying compiler arguments for check_demos
f63611dc87 artiq: Specify target CPU when creating LLVM target options
c86b6e22d6 standalone: Add command line flags for target properties
5722f3397e core: Switch to LLVM New Pass Manager
70cb0bd898 core: Add legacy_pm and target fields to CodeGenLLVMOptions
Compare 5 commits »
derppening commented on pull request M-Labs/nac3#319 2023-09-14 14:16:13 +08:00
Use LLVM New Pass Manager

On second thought - I think it's better if I implement a flag which can toggle between the new and old pass managers, since there are still outstanding bugs we need to fix when building using the…

derppening commented on pull request M-Labs/nac3#319 2023-09-14 14:13:05 +08:00
Use LLVM New Pass Manager

v3: Rebased against master, added target CPU name to TargetMachine creation, and updated check_demos.sh script to allow arguments to the compiler.

derppening pushed to llvm-new-pass-manager at M-Labs/nac3 2023-09-14 14:11:49 +08:00
6f9c464230 meta: Allow specifying compiler arguments for check_demos
75ae582565 artiq: Specify target CPU when creating LLVM target options
e86fae9941 standalone: Add command line flags for target properties
578fb56a78 core: Switch to LLVM New Pass Manager
ad834d2237 core: Add target field to CodeGenLLVMOptions
Compare 9 commits »
derppening commented on issue M-Labs/nac3#315 2023-09-12 19:04:36 +08:00
Incorrect Value when assigning list slices using OptimizationLevel::None

This issue seemed to have gone away using the new pass manager. I guess more investigation is needed to determine whether this is a LLVM bug or NAC3 bug.

derppening commented on pull request M-Labs/nac3#319 2023-09-12 18:51:42 +08:00
Use LLVM New Pass Manager

v2: Rebased against master.

derppening pushed to llvm-new-pass-manager at M-Labs/nac3 2023-09-12 18:51:24 +08:00
966ce7a659 standalone: Add command line flags for target properties
5ea7a672e6 core: Switch to LLVM New Pass Manager
894583eb13 core: Add target field to CodeGenLLVMOptions
cb3939634e core: Add CodeGenTargetMachineOptions
66f07b5bf4 flake: switch to nixos-unstable
Compare 7 commits »
derppening commented on pull request M-Labs/nac3#319 2023-09-12 18:33:14 +08:00
Use LLVM New Pass Manager

Do you understand why it was set to ""?

Not really. git blame appears to show that this change was made by you, so maybe you would have an idea?

What are the consequences of this…

derppening commented on pull request M-Labs/nac3#318 2023-09-12 16:21:50 +08:00
Add command-line parser and some debug options

Changed to use match expression.

derppening pushed to cmdline-parser at M-Labs/nac3 2023-09-12 16:20:57 +08:00
008d50995c meta: Update run_demo.sh
474f9050ce standalone: Expose flags in command-line
Compare 2 commits »
derppening created pull request M-Labs/nac3#319 2023-09-12 15:34:35 +08:00
WIP: Use LLVM New Pass Manager
derppening pushed to llvm-new-pass-manager at M-Labs/nac3 2023-09-12 15:14:03 +08:00
326ed52324 standalone: Add command line flags for target properties
27ccb0475c core: Switch to LLVM New Pass Manager
03cb907484 core: Add target field to CodeGenLLVMOptions
ae68b6c075 core: Add CodeGenTargetMachineOptions
Compare 4 commits »
derppening pushed to llvm-new-pass-manager at M-Labs/nac3 2023-09-12 14:53:14 +08:00
6a1a7b9a2b standalone: Add command line flags for target properties
a19828421d core: Switch to LLVM New Pass Manager
47e6216b24 core: Add target field to CodeGenLLVMOptions
059520ef97 artiq: Refactor to use CodegenTargetMachineOptions
2c3297f9cb core: Add CodeGenTargetOptions
Compare 6 commits »
derppening commented on pull request M-Labs/nac3#318 2023-09-12 14:49:01 +08:00
Add command-line parser and some debug options

I didn't notice that it was there. Removed the `

derppening pushed to cmdline-parser at M-Labs/nac3 2023-09-12 14:48:31 +08:00
37402a8d37 meta: Update run_demo.sh
derppening pushed to llvm-new-pass-manager at M-Labs/nac3 2023-09-12 14:41:40 +08:00
2f9c65bb6e standalone: Add command line flags for target properties
derppening pushed to llvm-new-pass-manager at M-Labs/nac3 2023-09-12 11:34:03 +08:00
400def7ac4 core: Switch to LLVM New Pass Manager
731d44d352 core: Add target field to CodeGenLLVMOptions
5c1e71dd05 artiq: Refactor to use CodegenTargetMachineOptions
9af4cfa696 core: Add CodeGenTargetOptions
3d7ccd0b69 meta: Update run_demo.sh
Compare 16 commits »
derppening commented on pull request M-Labs/nac3#318 2023-09-12 11:04:36 +08:00
Add command-line parser and some debug options

This merge request should be ready for re-review.