[PATCH] D109513: [AsmPrinter, SystemZ] Allow target to add instructions before section is ended.
Jonas Paulsson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Sep 10 05:06:36 PDT 2021
jonpa added a comment.
In D109513#2994113 <https://reviews.llvm.org/D109513#2994113>, @uweigand wrote:
> In D109513#2994077 <https://reviews.llvm.org/D109513#2994077>, @jonpa wrote:
>
>> In D109513#2992888 <https://reviews.llvm.org/D109513#2992888>, @aeubanks wrote:
>>
>>> do you think it'd make sense to merge emitConstantPools into emitSectionEndings?
>>
>> How about moving emitConstantPools() into the default implementation of emitSectionEndings() like this..?
>
> Hmm ... that makes me wonder: can we instead just use the existing emitConstantPools in the SystemZ back end to output the EXRL instructions? They are in effect a form of constants ... That shouldn't require any common code changes then.
I read in the original bug report: "...adding the XC instructions in some other text section out of the eyes of debug is not a good option for z/OS (details excluded)". Therefore I thought they better be placed in the text section...
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109513/new/
https://reviews.llvm.org/D109513
More information about the llvm-commits
mailing list