Clang uses LLVM’s integrated assembler by default on most targets, however non-integrated-as mode is default on AIX.
Currently integrated-as mode on AIX has passed tests of LLVM test-suite, bootstrap and Spec2017, therefore this patch sets integrated-as as the default assembler mode on AIX.
Details
Details
- Reviewers
shchenz DiggerLin hubert.reinterpretcast daltenty - Group Reviewers
Restricted Project - Commits
- rG533af6e5c4b3: [AIX] make integrated-as as default on AIX.
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
llvm/lib/MC/MCAsmInfoXCOFF.cpp | ||
---|---|---|
55 ↗ | (On Diff #522919) | Instead of setting the value to true you can just remove the line all together. |
llvm/lib/MC/MCAsmInfoXCOFF.cpp | ||
---|---|---|
55 ↗ | (On Diff #522919) | +1 And maybe this default value change can be put into another patch, as this is for backend? |
clang/test/Driver/aix-integrated-as.c | ||
---|---|---|
10 | Thanks, it seems an unnecessary check. |
both -fno-integrated-as and -fintegrated-as output the same file name "aix-integrated-as.o" , do you want to check "aix-integrated-as.o" here