[PATCH] D93769: [clang] Add support for option -ffp-eval-method and extend #pragma float_control similarly

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 20 13:39:01 PDT 2021


MaskRay added a comment.

In D93769#2891549 <https://reviews.llvm.org/D93769#2891549>, @mibintc wrote:

> In D93769#2891545 <https://reviews.llvm.org/D93769#2891545>, @MaskRay wrote:
>
>> This breaks `clang/test/Index/preamble-reparse-changed-module.m`
>>
>> http://45.33.8.238/linux/51638/step_7.txt
>
> I saw this fail on console when i do "ninja check-clang" but when I run each step in this test separately there is no failure
> the return code is 0
> I don't understand what the failure is, how can I discover the problem?  Thank you.  I can revert the patch

Seems like a libclang crash (an `assert` failure) if you enable LLVM_ENABLE_ASSERTIONS=on.

You can reproduce with check-clang-index


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93769/new/

https://reviews.llvm.org/D93769



More information about the cfe-commits mailing list