[PATCH] D147481: [M68k] Add basic Clang supports for M68881/2

2023-04-24 Thread Min-Yih Hsu via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rG9b617081420d: [M68k] Add basic Clang support for M68881/2 (authored by myhsu). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147481/new/ https://reviews.llv

[PATCH] D147481: [M68k] Add basic Clang supports for M68881/2

2023-04-23 Thread John Paul Adrian Glaubitz via Phabricator via cfe-commits
glaubitz added a comment. @myhsu Can you fix the typo in the commit message? s/supports/support/ CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147481/new/ https://reviews.llvm.org/D147481 ___ cfe-commits mailing list cfe-commits@lists.llvm.o

[PATCH] D147481: [M68k] Add basic Clang supports for M68881/2

2023-04-22 Thread Sheng via Phabricator via cfe-commits
0x59616e added a comment. LGTM. Thanks ! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147481/new/ https://reviews.llvm.org/D147481 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-co

[PATCH] D147481: [M68k] Add basic Clang supports for M68881/2

2023-04-22 Thread Min-Yih Hsu via Phabricator via cfe-commits
myhsu added a comment. ping CHANGES SINCE LAST ACTION https://reviews.llvm.org/D147481/new/ https://reviews.llvm.org/D147481 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D147481: [M68k] Add basic Clang supports for M68881/2

2023-04-06 Thread Min-Yih Hsu via Phabricator via cfe-commits
myhsu updated this revision to Diff 511477. myhsu marked 2 inline comments as done. myhsu added a comment. - Explicit add `isa-68882` even for newer (>=68040) CPUs. Such that's it's a lot easier to conditionally add related macro definitions with TargetOptions::FeatureMap - Add cases to test the

[PATCH] D147481: [M68k] Add basic Clang supports for M68881/2

2023-04-04 Thread Simon Pilgrim via Phabricator via cfe-commits
RKSimon added inline comments. Comment at: clang/test/Driver/m68k-macros.cpp:2 // Check macro definitions // RUN: %clang -target m68k-unknown-linux -m68000 -dM -E %s | FileCheck --check-prefix=CHECK-MX %s +// RUN: %clang -target m68k-unknown-linux -m68000 -mhard-float -dM -E %

[PATCH] D147481: [M68k] Add basic Clang supports for M68881/2

2023-04-03 Thread Min-Yih Hsu via Phabricator via cfe-commits
myhsu created this revision. myhsu added reviewers: 0x59616e, RKSimon. Herald added a project: All. myhsu requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. - Add the `-m68881` flag - Add floating point feature detection - Macro defini