Skip to content
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

Turn on ARM Integrated Assembler by default #18532

Closed
rengolin opened this issue Dec 6, 2013 · 5 comments
Closed

Turn on ARM Integrated Assembler by default #18532

rengolin opened this issue Dec 6, 2013 · 5 comments
Assignees
Labels
bugzilla Issues migrated from bugzilla

Comments

@rengolin
Copy link
Member

rengolin commented Dec 6, 2013

Bugzilla Link 18158
Resolution FIXED
Resolved on Dec 11, 2013 05:09
Version unspecified
OS Linux
Depends On #18454

Extended Description

In order to simplify execution, reduce external tool-chain dependencies and facilitate the cross-compilation process, we propose to enable the integrated assembler by default.

There were a few critical bugs open that would stop this from happening, but they were all fixed by now, so we propose that, for the remaining non-critical bugs, we allow a -no-integrated-as flag.

@rengolin
Copy link
Member Author

rengolin commented Dec 6, 2013

assigned to @rengolin

@rengolin
Copy link
Member Author

rengolin commented Dec 6, 2013

Bug #​18080 was the last of the required ones by my count.

Other bugs that will need to be implemented to make the IAS complete are:
#​8721
#​15172
#​17769
#​18019

But they're not required to turn the IAS on by default.

@rengolin
Copy link
Member Author

rengolin commented Dec 6, 2013

Checking on the LLVM Linux list for possible problems...

@rengolin
Copy link
Member Author

Build systems call the compiler to assemble files, so we need to support some of the assembler flags GCC does to make a seamless change between GAS and Clang:

clang-3.5: error: unsupported argument '-funwind-tables' to option 'Xassembler'
clang-3.5: error: unsupported argument '-march=armv7-a' to option 'Wa,'

@rengolin
Copy link
Member Author

Changed in r197024

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 9, 2021
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla
Projects
None yet
Development

No branches or pull requests

1 participant