[PATCH] D109950: [Clang] Enable IC/IF mode for __ibm128
Elizabeth Andrews via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Nov 1 18:06:25 PDT 2021
eandrews added a comment.
In D109950#3097652 <https://reviews.llvm.org/D109950#3097652>, @rjmccall wrote:
> In D109950#3097544 <https://reviews.llvm.org/D109950#3097544>, @eandrews wrote:
>
>> In D109950#3097161 <https://reviews.llvm.org/D109950#3097161>, @rjmccall wrote:
>>
>>> Oh, yes, I think this should be preserving the old logic there and just adding a new clause for explicit requests for ibm128, right?
>
> I do wonder whether that's the right priority between `float128_t` and a double-double `long double`. I think that particular configuration only exists on certain PPC targets; does anyone know what GCC does?
I don't know what GCC does and am not really sure how to go about figuring it out. I've uploaded a patch preserving old logic for review here - https://reviews.llvm.org/D112975. If I should do something different, please let me know. Thank you for your input!
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D109950/new/
https://reviews.llvm.org/D109950
More information about the cfe-commits
mailing list