This revision was automatically updated to reflect the committed changes.
Closed by commit rL366027: [clang][Driver][ARM] Favor -mfpu over default CPU
features (authored by alelab01, committed by ).
Herald added a subscriber: kristina.
Changed prior to commit:
https://reviews.llvm.org/D63936?vs
labrinea added inline comments.
Comment at: llvm/lib/Support/ARMTargetParser.cpp:412
- if (Extensions & AEK_CRC)
-Features.push_back("+crc");
- else
-Features.push_back("-crc");
-
- if (Extensions & AEK_DSP)
-Features.push_back("+dsp");
- else
-Features.push
ostannard added inline comments.
Comment at: llvm/lib/Support/ARMTargetParser.cpp:412
- if (Extensions & AEK_CRC)
-Features.push_back("+crc");
- else
-Features.push_back("-crc");
-
- if (Extensions & AEK_DSP)
-Features.push_back("+dsp");
- else
-Features.pus
labrinea added inline comments.
Comment at: llvm/lib/Support/ARMTargetParser.cpp:412
- if (Extensions & AEK_CRC)
-Features.push_back("+crc");
- else
-Features.push_back("-crc");
-
- if (Extensions & AEK_DSP)
-Features.push_back("+dsp");
- else
-Features.push
ostannard added inline comments.
Comment at: llvm/lib/Support/ARMTargetParser.cpp:412
- if (Extensions & AEK_CRC)
-Features.push_back("+crc");
- else
-Features.push_back("-crc");
-
- if (Extensions & AEK_DSP)
-Features.push_back("+dsp");
- else
-Features.pus
labrinea updated this revision to Diff 207436.
labrinea added a comment.
I've split the patch.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D63936/new/
https://reviews.llvm.org/D63936
Files:
clang/lib/Driver/ToolChains/Arch/ARM.cpp
clang/test/
