[PATCH] D135097: Remove redundant option '-menable-unsafe-fp-math'.

Yi Kong via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Oct 15 07:21:16 PDT 2022


kongyi added a comment.

In D135097#3859917 <https://reviews.llvm.org/D135097#3859917>, @jrtc27 wrote:

> It also will fail if the configured default target triple is one where the default program address space is non-zero, and maybe other things (do any targets do pre-IR name mangling for C?)

Our internal builder is broken due to this failing test, and I took the liberty to fix the test in commit f118280b04 <https://reviews.llvm.org/rGf118280b0407c40b1aacf40ad6777d777e333c88> by relaxing the variable name check. Feel free to change it if you want a different fix.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D135097



More information about the cfe-commits mailing list