https://github.com/vhscampos commented:

Review of the commit msg:
- Add some categories to the title, like [Clang][ARM]
- The new solution still uses the command-line options (inside the function 
`getARMTargetFeatures`). However it also takes into account the architecture in 
order to determine the alignment access policy.
- I think "bypass" is the wrong term. One correct term I believe could be "pass 
along", as in "Clang passes along the options to the multilib system".

https://github.com/llvm/llvm-project/pull/134099
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to