[PATCH] D46364: Add basic compiler-rt builtins support for hexagon

Krzysztof Parzyszek via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 3 12:53:22 PDT 2018


kparzysz requested changes to this revision.
kparzysz added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/builtins/hexagon/dfaddsub.S:404
+	// adding infinities is only OK if they are equal
+#if (__HEXAGON_ARCH__ == 61)
+	{
----------------
sidneym wrote:
> kparzysz wrote:
> > This is always false.  Please remove these checks.
> I left them here in the event this library is ever used for a v61 target.
That will never happen.


Repository:
  rCRT Compiler Runtime

https://reviews.llvm.org/D46364





More information about the llvm-commits mailing list