[clang] [llvm] [MC] Remove UseAssemblerInfoForParsing (PR #91082)
Peter Smith via cfe-commits
cfe-commits at lists.llvm.org
Tue May 7 07:31:40 PDT 2024
https://github.com/smithp35 commented:
The code-changes look good, what I'm less sure about is whether this is the right thing to do unconditionally.
A few months ago I would likely to have said this wouldn't matter that much as most people don't use `-S` in there build systems so any failures are likely to have been of minor incovenience. However we recently encountered a customer using `-save-temps` on their main build, which produces assembly and then re-assembles it; in our case there was a problem with describing the target to the assembler so that step failed.
With this in mind I'm a bit reluctant to say that this wouldn't affect anyone's use case.
This could be worth a RFC on Discourse to see if there is wider support or just apathy?
https://github.com/llvm/llvm-project/pull/91082
More information about the cfe-commits
mailing list